Bug 4395 - locale strings are mistaken for uris
Summary: locale strings are mistaken for uris
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Plugins (show other bugs)
Version: 3.0.4
Hardware: Other other
: P5 minor
Target Milestone: Undefined
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords: triage
Depends on:
Blocks:
 
Reported: 2005-06-07 03:28 UTC by Bas Zoetekouw
Modified: 2005-08-18 01:44 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
testcase email text/plain None Bas Zoetekouw [NoCLA]
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 Bas Zoetekouw 2005-06-07 03:28:45 UTC 
locale strings that occur in message bodies are mistaken for mailto urls by
URIDNSBL:

debug: plugin: Mail::SpamAssassin::Plugin::URIDNSBL=HASH(0x859391c) implements
'parsed_metadata'
debug: dns_available set to yes in config file, skipping test
debug: uri found: mailto:LC_CTYPE=fr_FR@euro
debug: uri found: mailto:LANG=fr_FR@euro
Comment 1 Bob Menschel 2005-07-13 23:56:38 UTC 
Triage:  Bas, from other discussions I think this might be fixed in 3.1.0. 
Could you attach an email to this bug entry that can be used to replicate the
problem, so I can test to see if it has or hasn't been fixed?  Thanks.
Comment 2 Bob Menschel 2005-07-14 00:10:14 UTC 
Triage: possibly related to bug 4449
Comment 3 Bas Zoetekouw 2005-07-14 01:05:46 UTC 
Created attachment 3023 [details]
testcase email

test case email.  The LANG=fr_FR@euro and LC_CTYPE=fr_FR@euro in the final line
are detected as URIs
Comment 4 Bob Menschel 2005-07-16 18:53:42 UTC 
Triage: When I run this against svn, the locale string is NOT treated as a URI.
Looks like this is fixed in 3.1.0.  

Bas, can you download the pre4 tar via
> wget http://people.apache.org/~jm/devel/Mail-SpamAssassin-3.1.0-pre4.tar.gz
expand it into its own directory, do the make, and NOT the make install, and
then from its top directory use the command
> ./spamassassin <testcase.email.txt >testcase.email.out 
and see if your problem goes away?  If so, I'll flag this as fixed.  Thanks.
Comment 5 Bas Zoetekouw 2005-08-17 23:42:21 UTC 
Yep, this seems fixed in 3.1.0-rc4
Comment 6 Bas Zoetekouw 2005-08-18 00:07:35 UTC 
uh, make that -pre4
Comment 7 Warren Togami 2005-08-18 02:16:48 UTC 
How common are these locale strings, and what negative effect can this
misdetection have?  (I am wondering if this is worthwhile to target at 3.0.5.)
Comment 8 Bas Zoetekouw 2005-08-18 02:53:09 UTC 
> How common are these locale strings, and what negative effect can this
> misdetection have?  (I am wondering if this is worthwhile to target at 3.0.5.)

I don't think they are very common.  The debian bug reporting tool includes them
in the mails it sends, but I can't think of any other occurrence of these locale
things.

Anyway, the worst thing that can happen, is that extra (bogus) uris are passed
to the URIDNSBL plugin.  Not worth backporting, IMO.
Comment 9 Justin Mason 2005-08-18 09:44:48 UTC 
marking FIXED