Page MenuHomeSoftware Heritage

Add sentry integration to swh-web
ClosedPublic

Authored by vlorentz on Dec 10 2019, 2:06 PM.

Diff Detail

Event Timeline

swh/web/browse/views/revision.py
285

note that I changed this to be stricted. But this block is wrapped in a larger except Exception as exc:, which will catch other exceptions, if any.

swh/web/common/throttling.py
70–106

I did some refactoring here to capture only ImproperlyConfigured instead of all exceptions; and also to make the try block as short as possible to not catch too many errors.

  • Work around pip's lack of a dependency resolver
  • Silence mypy about override of post_fork

add tests for gunicorn_config.

anlambert added inline comments.
swh/web/settings/gunicorn_config.py
1

The swh/web/settings folder is dedicated to contain Django configuration.

I think that file should be moved in the swh/web directory which already contains configuration related modules.

This revision is now accepted and ready to land.Dec 10 2019, 2:57 PM
This revision was landed with ongoing or failed builds.Dec 10 2019, 5:58 PM
This revision was automatically updated to reflect the committed changes.