Bug 6043 - get_uri_detail_list should export "linkify" status for parsed URIs
Summary: get_uri_detail_list should export "linkify" status for parsed URIs
Status: NEW
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Libraries (show other bugs)
Version: 3.2.5
Hardware: All All
: P5 enhancement
Target Milestone: 4.0.0
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-12 17:55 UTC by Paul Fisher
Modified: 2019-08-02 12:13 UTC (History)
1 user (show)



Attachment Type Modified Status Actions Submitter/CLA Status

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Fisher 2009-01-12 17:55:01 UTC
Per Spamhaus recommendations, we run our our SpamAssassin configuration with URIBL_SBL set to a higher than default value.  We recently had a false positive because an emailed diff contained the text "configure.ac".  "configure.ac" contains a valid TLD and it has NS records which are currently listed on the SBL.

While one could categorize this issue to a local configuration problem, it might be worthwhile to add some functionality to get_uri_detail_list in PerMsgStatus so plugins can make more intelligent decisions about the kinds of URIs contained in an email.

The SpamAssassin parse engine in the SVN tree knows which kinds of parsed URIs are linkified via $rblonly ("configure.ac" obviously isn't), but this data isn't exported for modules such as URIDNSBL to use.

If this functionality were available, one could assign the default (low) value to URIBL_SBL for parsed non-linkified URIs, and a high value for all other URIs (parsed linkified and within html) via the existing plug-in framework.
Comment 1 Henrik Krohns 2019-08-02 12:13:31 UTC
Such flag (types->unlinked) will be in 4.0.0, but I'm not sure if it would be wise to modify scored based on it. Will leave this open for comments while I work on it.