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

--- Comment #15 from Mark Martinec <[email protected]> 2010-03-02 18:54:03 
UTC ---
> Qmail not allow email with capitals letter, it automatic lowercase all
> cheracters and I forgot in regexp.

Does it make a syntax error when a capital letter is present?
If not, it should be allowed. The only purpose of that regexp is parsing,
not e-mail address syntax validation.

> About other characters I not sure. I not know any top level domain with 
> number.

Here is one: [email protected]

See:
  https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6353#c10

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

A TLD can have more than 4 characters, e.g.:  .museum, .travel .
See:
  http://en.wikipedia.org/wiki/Sponsored_top-level_domain

Do not be too strict there, the only purpose is to parse that config line.
Perhaps even splitting on whitespace might suffice.

> 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.

Ok, thanks, will apply that on the next round.

-- 
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