Skip to content

fix: bound DoH error excerpts - #938

Merged
ryanfowler merged 1 commit into
mainfrom
audit-9-doh-error-body
Aug 5, 2026
Merged

fix: bound DoH error excerpts#938
ryanfowler merged 1 commit into
mainfrom
audit-9-doh-error-body

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • keep the existing 1 MiB DoH response body bound
  • cap DoH error excerpts at 4 KiB with a truncation marker
  • escape DoH error excerpts with shared terminal-safe escaping
  • add oversized and control-character tests

Audit finding 9 only.

@ryanfowler
ryanfowler enabled auto-merge (squash) August 5, 2026 15:43
@ryanfowler
ryanfowler merged commit b20c946 into main Aug 5, 2026
4 checks passed
@ryanfowler
ryanfowler deleted the audit-9-doh-error-body branch August 5, 2026 15:51
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