Bug 8109 - DecodeShortURLs should support refresh header
Summary: DecodeShortURLs should support refresh header
Status: NEW
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Plugins (show other bugs)
Version: 4.0.0
Hardware: PC Linux
: P3 enhancement
Target Milestone: Undefined
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-01-20 13:53 UTC by Christer Mjellem Strand
Modified: 2023-01-20 13:54 UTC (History)
1 user (show)


Attachment Type Modified Status Actions Submitter/CLA Status
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Christer Mjellem Strand 2023-01-20 13:53:10 UTC 
DecodeShortURLs will already parse the location header, which is expected. However, some redirectors will instead use the refresh header, which will work well in a browser, but not so much with the plugin.

One example of this is the marketing redirector public.heyloyalty.com, frequently seen in newsletters. It will return a 302, but only with a refresh header, no location header.

Example response:

refresh: 0; url=http://example.com/

(Original and target URLs omitted for privacy reasons)

It would be nice if the plugin was able to also parse, process and follow this.