]> git.ipfire.org Git - thirdparty/squid.git/commit
Bug 3390: Proxy auth data visible to scripts (#2249)
authorAmos Jeffries <yadij@users.noreply.github.com>
Sat, 11 Oct 2025 03:33:02 +0000 (16:33 +1300)
committerGitHub <noreply@github.com>
Sat, 11 Oct 2025 03:33:02 +0000 (16:33 +1300)
commite7e9073a2435cc93b913553d147b497fda77c1ab
tree677f1485f81e35612dbc04e1c128bfb907912ce2
parentbbc23aada0cde2712e0a8fdc35598650007638df
Bug 3390: Proxy auth data visible to scripts (#2249)

Original changes to redact credentials from error page %R code
expansion output was incomplete. It missed the parse failure
case where ErrorState::request_hdrs raw buffer contained
sensitive information.

Also missed was the %W case where full request message headers
were generated in a mailto link. This case is especially
problematic as it may be delivered over insecure SMTP even if
the error was secured with HTTPS.

After this change:
* The HttpRequest message packing code for error pages is de-duplicated
  and elides authentication headers for both %R and %W code outputs.
* The %R code output includes the CRLF request message terminator.
* The email_err_data directive causing advanced details to be added to
  %W mailto links is disabled by default.

Also redact credentials from generated TRACE responses.

---------

Co-authored-by: Alex Rousskov <rousskov@measurement-factory.com>
doc/release-notes/release-7.sgml.in
src/HttpRequest.cc
src/HttpRequest.h
src/cf.data.pre
src/client_side_reply.cc
src/errorpage.cc
src/errorpage.h
src/tests/stub_HttpRequest.cc