https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8375
Kent Oyer <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #1 from Kent Oyer <[email protected]> --- Fixed in r1932298. Rewrite AuthRes plugin to use new Header::AuthenticationResults parser The AuthRes plugin previously hand-rolled a fragile A-R header parser that died (discarding the entire header) on unknown methods or properties. Real-world headers from servers like Mail::Milter::Authentication include extended properties (policy.published-domain-policy) and non-standard methods (x-tls) that caused valid results to be skipped. Create Mail::SpamAssassin::Header::AuthenticationResults, inheriting from ParameterHeader, which handles the structural parsing (comment stripping, semicolon tokenization, quoted strings, line unfolding). The subclass adds A-R-specific secondary parsing of method results and ptype.property=value pairs. Simplify the AuthRes plugin to use the new class. Remove the hand-rolled parser (regex constants, skip_cfws, the old parse_authres), the method/ property whitelists, and ARC handling (handled in other plugins). All parsed methods and properties now flow through to $pms->{authres_parsed} regardless of whether they appear in a whitelist, so check_authres_result works for any method present in the header. The $pms->{authres_parsed} and $pms->{authres_result} structures are preserved for compatibility with other plugins. -- You are receiving this mail because: You are the assignee for the bug.
