https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8441
Bug ID: 8441
Summary: bad shortener entry te.ee doesn't work and breaks test
Product: Spamassassin
Version: SVN Trunk (Latest Devel Version)
Hardware: All
OS: All
Status: NEW
Severity: minor
Priority: P2
Component: Rules
Assignee: [email protected]
Reporter: [email protected]
Target Milestone: Undefined
The recent commit of tr.ee in rules/25_url_redirectors.cf (r1939251) caused
t/all_modules.t to fail by surfacing an existing error in the tr.ee entry in
rules/25_url_redirectors.cf
The tr.ee shortener only works using get, not head, so it never worked declared
with url_shortener, which uses head. When it was added as a get url_redirector,
the mismatch head vs get caused the test failure in trunk.
In 4.0.2, where shorteners and redirectors have different code, the tr.ee
shortener links silently aren't detected.
I'm fixing it by changing the entry in rules/25_url_redirectors.cf to use
url_shortener_get which lets the shortener plugin work and eliminates the test
failure.
Since this change is in a rule update, I'll commit it without a vote, but I
wanted to document this in Bugzilla in case other shortener sites turn out to
require get instead of head.
--
You are receiving this mail because:
You are the assignee for the bug.
