Skip to content

Protect exception reports from sensitive data. - #96

Merged
samuel-williams-shopify merged 2 commits into
mainfrom
protect-exception-reports
Aug 17, 2026
Merged

Protect exception reports from sensitive data.#96
samuel-williams-shopify merged 2 commits into
mainfrom
protect-exception-reports

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • redact sensitive request headers, query parameters, session state, and controller variables by default
  • omit the raw query string from the report request line
  • make request body attachments opt-in with dump_body: true
  • bound body and environment attachments with attachment_size_limit
  • make dump_environment: false exclude application state from the report body

Verification

  • bundle exec sus test/utopia/exceptions/mailer.rb (15 tests, 47 assertions)
  • bundle exec rubocop lib/utopia/exceptions/mailer.rb test/utopia/exceptions/mailer.rb
  • bundle exec bake decode:index:coverage lib
  • git diff --check

The full local suite reached 482 passing tests. Its four command-generation tests remain blocked by the existing stale vendor/bundle native extensions and unavailable locally installed protocol-url dependency; CI runs those tests with a clean bundle.

Assisted-By: devx/cbdeae41-9308-4071-ad53-58cd92db2946
Assisted-By: devx/cbdeae41-9308-4071-ad53-58cd92db2946
@samuel-williams-shopify
samuel-williams-shopify merged commit 3ee7c02 into main Aug 17, 2026
36 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the protect-exception-reports branch August 17, 2026 08:19
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.

1 participant