Skip to content

Upgrade sentry-symfony to version 4.2

Fridtjof requested to merge sentry-upgrade into master

What does this MR do?

Just updates the sentry-symfony integration (from 3.5 to 4.2). This will enable new features (like performance tracing).

Note that tracing is not enabled yet, for that we will have to set the sample rate. I think it would be a good idea to set different sample rates depending on beta vs prod, but that's for later.

How confident are you it won't break things if deployed?

I followed the guide at https://github.com/getsentry/sentry-symfony/blob/master/UPGRADE-4.0.md. Still, it might break in prod - dev obviously does not send messages to sentry.io. Of course, I checked that the website still runs.

Let's try it in beta!

Links to related issues

How to test

  1. check out locally
  2. update packages
  3. check the website still works

Checklist

  • no test needed, just a dependency bump
  • no unrelated changes
  • asked someone for a code review
  • set a "for:" label to indicate who will be affected by this change
  • used "state:" labels to track this MR's state until it was beta tested
  • added an entry to CHANGELOG.md
  • added a short text that can be used in the release notes
  • Once your MR has been merged, you are responsible to create a testing issue in the Beta Testing forum: https://foodsharing.de/?page=bezirk&bid=734&sub=forum
    • Consider writing a detailed description in German.
    • Describe in a few sentences, what should be tested from a user perspective.
    • Also mention different settings (e.g. different browsers, roles, ...) how this change can be tested.
    • Be aware, that also non technical people should understand.
Edited by Fridtjof

Merge request reports