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

Downgrade Airbrake and use standard config #438

Merged
merged 1 commit into from Aug 11, 2017
Merged

Downgrade Airbrake and use standard config #438

merged 1 commit into from Aug 11, 2017

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Aug 11, 2017

We’re seeing some errors with Airbrake 5:
Airbrake: falling back to sync delivery because there are no running async workers
Airbrake: unexpected code (200). Body: {“notice”: … }

Also fixes other incompatibilities outlined in
https://docs.publishing.service.gov.uk/manual/upgrade-rails.html#airbrake

Downgrades to version 4 with custom branch we are using along with default config:
https://github.com/alphagov/govuk-rails-app-template/blob/master/templates/config/initializers/airbrake.rb

See also:
errbit/errbit#1087 (comment)

We’re seeing some errors with Airbrake 5:
`Airbrake: unexpected code (200). Body: {“notice”: … }`

Also fixes other incompatibilities outlined in
https://docs.publishing.service.gov.uk/manual/upgrade-rails.html#airbrak
e

See:
errbit/errbit#1087 (comment)
@boffbowsh boffbowsh temporarily deployed to government-frontend-pr-438 Aug 11, 2017 Inactive
@fofr fofr merged commit c2519bd into master Aug 11, 2017
2 checks passed
2 checks passed
continuous-integration/jenkins/branch This commit looks good
Details
security/snyk No new vulnerabilities
Details
@fofr fofr deleted the use-old-airbrake branch Aug 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.