https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6340

Sossi Andrej <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #14 from Sossi Andrej <[email protected]> 2010-03-02 18:34:00 UTC 
---
(In reply to comment #13)
> > Created an attachment (id=4686)
 --> (https://issues.apache.org/SpamAssassin/attachment.cgi?id=4686) [details]
[details]
> > regexp correction
> 
> Which is (compared to the already applied changes):
> 
> -  } elsif ($vpopusername =~ /\@/) {
> +  } elsif ($vpopusername =~ /^[a-z0-9._%+-...@[a-z0-9.-]+\.[a-z]{2,4}$/) {
> 
> What about capitals in the email address?
> In principle any character is allowed in a mailbox name, as long as it is
> in double quotes. And the IDN ccTLDs allow a '-' and digits in the name.

Qmail not allow email with capitals letter, it automatic lowercase all
cheracters and I forgot in regexp. Sorry. About other characters I not sure. I
not know any top level domain with number.
Qmail not allow double quotes or other special character in forward line. From
documentation:
[...]
# the following examples are WRONG
&m...@new
&<[email protected]>
& [email protected]
&[email protected] (New Address)
[...]

I tested also
&"[email protected]"
and qmail said:
Sorry, I couldn't find any host named new.job.com". (#5.1.2)


Corrected regexp:
    } elsif ($vpopusername =~ /^[^#\| ][^\@ ]...@[^\@ ]+\.[a-zA-Z]{2,4}$/) {

and correct also:
-        if ($vpopusername =~ s{(/.+)/Maildir/}{$1}) {
+        if ($vpopusername =~ s{^(/.+)/Maildir/$}{$1}) {
Previously regexp match also:
#/usr/local/vpopmail/domains/foo/bar
but it is a comment.

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to