Skip to content

C
coopwire-hypothesis
Switch branch/tag
  1. 27 Aug, 2019 5 commits
  3. 26 Aug, 2019 5 commits
  5. 23 Aug, 2019 5 commits
    • Kyle Keating's avatar
      Namespace selection module · 7c2eea8e
      Kyle Keating authored 
      - Move selectedTab logic out of annotations reducer to selection reducer
- removeAnnotations action now passes the set of removed annotations and remaining annotations
- buildRootThread takes getRootState param
      7c2eea8e
    • Hannah Stepanek's avatar
      Merge pull request #1320 from hypothesis/use-new-sentry-sdk · 5552dbfc
      Hannah Stepanek authored 
      Replace legacy Sentry JS SDK with new Sentry SDK
      5552dbfc
    • Robert Knight's avatar
      Support configuring the Sentry environment · cac16aa0
      Robert Knight authored 
      This enables us to distinguish reports from dev, qa, prod etc. The value
is provided by h for the embed or the browser extension.
      cac16aa0
    • Robert Knight's avatar
      Implement client-side rate limiting per session · fd022199
      Robert Knight authored 
      Previous experience with Sentry has found that there are various
scenarios that can cause a client to spam the server with a large number
of reports. Although Sentry has its own server-side quotas and rate
limiting, I think it will be useful to do client-side per-session rate
limiting as well. This also provides a place where we can hook in other
client-side filtering in future.
      fd022199
    • Robert Knight's avatar
      Replace raven-js with Sentry's new JavaScript SDK · 542b96f0
      Robert Knight authored 
      Replace the legacy raven-js package with Sentry's new JS SDK [1], with a
minimal wrapper in `util/sentry.js` that exports the same interface as
the previous `raven.js` module.

To enable Sentry, h or the browser extension must set a `sentry` key in the
sidebar app's configuration.

As part of switching SDKs, the following changes were made:

 - The AngularJS integration was removed, as we are in the process of
   migrating away from Angular
 - The custom code for unhandled promise rejections was removed, as the
   new SDK handles this already
 - The URL rewriting logic was removed. This was done to make the setup
   as minimal as possible. This may need to be re-added later.
 - The `report` function was removed, as it was not used anywhere
 - The `session` module now imports `util/sentry` directly rather than
   using Angular dependency injection. Now that we have a convenient way
   to mock all CJS/ES imports, there isn't a need to use DI for
   utilities

[1] https://docs.sentry.io/platforms/javascript/
      542b96f0
  7. 22 Aug, 2019 4 commits
  9. 21 Aug, 2019 4 commits
  11. 20 Aug, 2019 7 commits
  13. 19 Aug, 2019 6 commits
  15. 15 Aug, 2019 4 commits