diff options
| author | Danilo M. <danix@danix.xyz> | 2026-09-09 19:07:08 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-09-09 19:07:08 +0200 |
| commit | 75d9744a53acf091e8a2a152210647773db189d1 (patch) | |
| tree | 8f4d37a52810a037cfc75e27ce9a3736dfc57714 /docs/BACKLOG.md | |
| parent | b35d6e504719da97f8d2ef19b300d5d8ee45beb0 (diff) | |
| download | abusectl-75d9744a53acf091e8a2a152210647773db189d1.tar.gz abusectl-75d9744a53acf091e8a2a152210647773db189d1.zip | |
fix: strip the envelope recipient in every shape the grammar allows
The for-clause rule anchored on the clause terminator, which is strictly
stronger than RFC 5321 4.4: For sits inside Opt-info, so With, ID, Via or a
CFWS comment may legitimately follow it, and its ABNF is 1*( Path / Mailbox )
where Mailbox carries no angle brackets. Only the tidiest shape was stripped.
Four ordinary ones published the victim's address in the one header a report
reproduces verbatim, including "for <a@b> (envelope-from <c@d>);", which is
routine Exim and Sendmail output.
Anchored on the address instead. The envelope SENDER survives the cut, since
that is what the report is about, and the leftover whitespace, orphan
separator and emptied comment are tidied because the line is published to a
third party verbatim. Table-driven over all five shapes; reverting the regex
fails six of nine tests in the class.
Two accepted disclosures are now named in the spec's "Where the headers come
from" rather than living in a test comment: our own relay's hostname, which
the by and authserv-id clauses carry by construction, and attacker-controlled
free text in Subject and the From display name, which the whitelist does not
and should not filter. The second is asserted as a documented limit so it
reads as deliberate, and the manifest assertion now bars the obfuscated
you%40example.org spelling alongside the plain one.
case.create() seeds an empty headers block so an unparsed case has the same
shape as a parsed one. Additive, so FORMAT_VERSION is unchanged.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Xj1ayFRSUQ2u7cwb3S4axE
Diffstat (limited to 'docs/BACKLOG.md')
0 files changed, 0 insertions, 0 deletions
