broken address validation

When trying to send a message to an address that has a dot i

Wed, 11 Jun 2008 12:51:21 +0000

When trying to send a message to an address that has a dot in the name part which is not followed by a blank I get a address validation error: e.g. "a.a.a" There was an error sending your message: Validation failed for: a.a.a This should not happen, there is nothing wrong with the address. "a.a.a d" -> works "a.a.a d.d" -> works "a. a." -> works

This is a bug in PEAR::Mail, not in Horde or IMP.

Wed, 11 Jun 2008 20:24:09 +0000

This is a bug in PEAR::Mail, not in Horde or IMP.

> This is a bug in PEAR::Mail, not in Horde or IMP. FWIW,

Fri, 13 Jun 2008 06:06:08 +0000

> This is a bug in PEAR::Mail, not in Horde or IMP. FWIW, I don't see this with PEAR::Mail from CVS (1.2.0a2+). This might have fixed things: http://pear.php.net/bugs/bug.php?id=9137

>> This is a bug in PEAR::Mail, not in Horde or IMP. > > F

Sun, 15 Jun 2008 13:51:21 +0000

>> This is a bug in PEAR::Mail, not in Horde or IMP. > > FWIW, I don't see this with PEAR::Mail from CVS (1.2.0a2+). This > might have fixed things: > http://pear.php.net/bugs/bug.php?id=9137 Strange, it doesn't work here. "a.a" nospam@tugraz.at -> works "a.a" -> fails

Same here, that patch didn't fix it.

Sun, 15 Jun 2008 14:30:39 +0000

Same here, that patch didn't fix it.

But the error is still coming from the PEAR package, not fro

Sun, 15 Jun 2008 17:04:46 +0000

But the error is still coming from the PEAR package, not from Horde.

> But the error is still coming from the PEAR package, not f

Mon, 16 Jun 2008 06:20:59 +0000

> But the error is still coming from the PEAR package, not from Horde. Right, and it is good to know that you are one of the authors of that package ;-)

Changes have been made in CVS for this ticket: http://cvs.h

Mon, 16 Jun 2008 06:46:35 +0000

Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/MIME/MIME.php?r1=1.192&r2=1.193&ty=u

Changes have been made in CVS for this ticket: http://cvs.h

Mon, 16 Jun 2008 07:21:04 +0000

Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/MIME/MIME.php?r1=1.139.4.35&r2=1.139.4.36&ty=u

Fixed. We weren't quoting the display portion of the address

Mon, 16 Jun 2008 07:23:11 +0000

Fixed. We weren't quoting the display portion of the address string correctly if it contains a period (this wasn't required to be quoted by RFC 822, but is required under RFC 2822), so Mail_RFC822 was correctly reporting that the address was invalid.

> Fixed. We weren't quoting the display portion of the addre

Mon, 16 Jun 2008 07:51:46 +0000

> Fixed. We weren't quoting the display portion of the address string > correctly if it contains a period (this wasn't required to be quoted > by RFC 822, but is required under RFC 2822), so Mail_RFC822 was > correctly reporting that the address was invalid. Thanks, it works now!

> Right, and it is good to know that you are one of the auth

Mon, 16 Jun 2008 14:36:22 +0000

> Right, and it is good to know that you are one of the authors of that package ;-) Still no reason not to use the correct bug tracker. Which this turned out to be in this case, but the point remains.