Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps(sentry): upgrade Sentry to version 8 #17505

Closed
wants to merge 2 commits into from
Closed

Commits on Sep 25, 2024

  1. deps(sentry): upgrade Sentry to version 8

    Because:
     - Sentry 8 is the latest version
    
    This commit:
     - deletes some integration code since Sentry does it automatically
     - updates code to use the Sentry 8 API
     - updates a few proxyquire calls because they were cause module not
       found errors in CI
    chenba authored and dschom committed Sep 25, 2024
    Configuration menu
    Copy the full SHA
    83dc649 View commit details
    Browse the repository at this point in the history
  2. bug(auth): Fix test timeouts

    Becuase:
    - After upgrading sentry we were seeing test timeouts
    - There were some kinda weird patterns in place for these tests anyways
    - Test server wasn't getting cleaned up
    - Test server start up is taking longer (presumably because of sentry upgrade?)
    
    This Commit:
    - Moves before an after blocks next to each other so setup and tear down are easy to follow
    - Makes sure test server start / stop is done in before each blocks
    - Increase test timeouts to 60s when test server is used
    - Switches to async/await for before and after blocks, which feels less error prone
    dschom committed Sep 25, 2024
    Configuration menu
    Copy the full SHA
    0394075 View commit details
    Browse the repository at this point in the history