Skip to content

CP-12437 - Harden Airbrake and New Relic config for production#68

Merged
bernardodsanderson merged 2 commits into
mainfrom
CP-12437
Jun 10, 2026
Merged

CP-12437 - Harden Airbrake and New Relic config for production#68
bernardodsanderson merged 2 commits into
mainfrom
CP-12437

Conversation

@bernardodsanderson

Copy link
Copy Markdown
Collaborator
  • Fix Airbrake root_directory from /var/cpd/app to Rails.root.to_s
  • Add blocklist_keys to prevent sensitive params in error reports
  • Add AIRBRAKE_IGNORE filter for RoutingError, RecordNotFound, etc.
  • Bump newrelic_rpm from 9.17.0 to 9.24.0 (match ATS)
  • Enable distributed_tracing in production
  • Include ENV_NAME in production app_name for instance differentiation

- Fix Airbrake root_directory from /var/cpd/app to Rails.root.to_s
- Add blocklist_keys to prevent sensitive params in error reports
- Add AIRBRAKE_IGNORE filter for RoutingError, RecordNotFound, etc.
- Bump newrelic_rpm from 9.17.0 to 9.24.0 (match ATS)
- Add job.sidekiq.args.* attributes to New Relic traces
- Enable distributed_tracing in production
- Include ENV_NAME in production app_name for instance differentiation

@mikhael-rakauskas mikhael-rakauskas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks solid to me! I'm assuming the ignored list of airbrakes is due to those being handled elsewhere/by another process/copied from ATS verbatim?

@bernardodsanderson

Copy link
Copy Markdown
Collaborator Author

@mikhael-rakauskas - The Airbrake ignore list was adapted from ATS but trimmed to only the ones relevant to Docuseal (Docuseal doesn't have Sift::WorkflowJobError, Inbound::ReplyHandlerEmailBlockedError, etc.)

@bernardodsanderson bernardodsanderson merged commit f804bb6 into main Jun 10, 2026
4 of 5 checks passed
@bernardodsanderson bernardodsanderson deleted the CP-12437 branch June 10, 2026 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants