Summary | Sender validation fails for non-ascii letters |
Queue | IMP |
Queue Version | Git master |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | slusarz (at) horde (dot) org |
Requester | wrobel (at) horde (dot) org |
Created | 01/19/2011 (5282 days ago) |
Due | |
Updated | 01/20/2011 (5281 days ago) |
Assigned | 01/19/2011 (5282 days ago) |
Resolved | 01/20/2011 (5281 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
State ⇒ Resolved
Bug #9525: Switch header charsets if we can't encode properlyhttp://git.horde.org/horde-git/-/commit/0c59c79c42530ea086a137b065da74d6292704f7
$message->getHeaderCharset() returns 'us-ascii' which is probably
causing the problems. I'm replying to another message. Might that
result in assuming an incorrect character set here?
Horde_Mime::encodeAddress call above it the one that should quote it?
will pass validation (since it no longer contains non-7bit characters).
Added some tests to horde/Mime - both pass for me.
Bug #9525: Add Horde_Mime::encodeAddress() testshttp://git.horde.org/horde-git/-/commit/8696a5446ebb64c22233e6ec37f6379ae42eb98c
The validation fails on l.850 in imp/lib/Compose.php. Is the
Horde_Mime::encodeAddress call above it the one that should quote it?
the validation correctly passes.
So for whatever reason this character is being quoted when I send a
message but not when you send a message. We need to find where in IMP
this quoting is failing to occur (unit tests are unfortunately not
going to help us here :( ).
Bug #9525: Add validation testshttp://git.horde.org/horde-git/-/commit/d17e468f3b8c781100fad3bfadb305fdd4164b5d
diff --git a/framework/Mail/test/Horde/Mail/ParseTest.php
b/framework/Mail/test/Horde/Mail/ParseTest.php
index f095c1c..fec8988 100644
--- a/framework/Mail/test/Horde/Mail/ParseTest.php
+++ b/framework/Mail/test/Horde/Mail/ParseTest.php
@@ -189,4 +189,16 @@ class Horde_Mail_ParseTest extends
PHPUnit_Framework_TestCase
$this->assertEquals(count($res), 3);
}
+ public function test
Bug9525()+ {
+ $parser = new Horde_Mail_Rfc822();
+ $ob = $parser->parseAddressList(
+ 'ß <wrobel@pardus.de>',
+ array(
+ 'default_domain' => 'pardus.de',
+ 'nest_groups' => false,
+ 'validate' => true
+ )
+ );
+ }
}
Having a unit test now I assume I would be able to solve it myself. I
just wont find the time today anymore and in case you immediately spot
the problem I'm of course happy if I don't have to dive into the code :)
State ⇒ Feedback
[...]
To: =?utf-8?b?w58=?= <slusarz@curecanti.org>
Subject: Test
User-Agent: Internet Messaging Program (IMP) H4 (5.0-git)
[...]
Patch ⇒ No
State ⇒ Unconfirmed
Milestone ⇒
Queue ⇒ IMP
Summary ⇒ Sender validation fails for non-ascii letters
Type ⇒ Bug
Priority ⇒ 1. Low
general but when trying to send a message to
ß <wrobel@horde.org>
in the dynamic version of imp I get the error that sending of the
message failed with
Validation failed for: ß <wrobel@horde.org>