Skip to content

fix: redact handler stats URLs and report stream consumption honestly#21

Merged
ernolf merged 2 commits into
mainfrom
ernolf/fix/handler-stats-redaction
Jul 19, 2026
Merged

fix: redact handler stats URLs and report stream consumption honestly#21
ernolf merged 2 commits into
mainfrom
ernolf/fix/handler-stats-redaction

Conversation

@ernolf

@ernolf ernolf commented Jul 19, 2026

Copy link
Copy Markdown
Owner
  • fix: redact credentials from logged handler stats URLs
  • fix: derive stream consumption from EOF and report detached bodies as unknown

Redacts url and redirect_url in the curl handler stats before they reach the log (they bypassed redactUri() and leaked e.g. Google secret iCal hashes), derives stream_consumed from the read position instead of an explicit close(), and reports detached bodies (stream => true consumers like the webcal service) as unknown instead of claiming 0 bytes read.

ernolf added 2 commits July 19, 2026 10:41
Signed-off-by: ernolf <raphael.gradenwitz@googlemail.com>
… unknown

Signed-off-by: ernolf <raphael.gradenwitz@googlemail.com>
@ernolf
ernolf merged commit 1a30889 into main Jul 19, 2026
36 checks passed
@ernolf
ernolf deleted the ernolf/fix/handler-stats-redaction branch July 19, 2026 08:49
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