You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by "Stefano Bagnara (JIRA)" <ji...@apache.org> on 2008/06/19 19:27:44 UTC
[jira] Created: (GERONIMO-4136) Encoding coversion issues with
geronimo javamail
Encoding coversion issues with geronimo javamail
------------------------------------------------
Key: GERONIMO-4136
URL: https://issues.apache.org/jira/browse/GERONIMO-4136
Project: Geronimo
Issue Type: Bug
Security Level: public (Regular issues)
Components: mail
Affects Versions: 2.1.1
Reporter: Stefano Bagnara
Assignee: Rick McGuire
We have a test trying to add a footer to a message using iso-8859-15 charset:
{code:title=AddFooterTest.java}
public void testAddFooterTextPlain() throws MessagingException, IOException {
// quoted printable mimemessage text/plain
String asciisource = "Subject: test\r\nContent-Type: text/plain; charset=ISO-8859-15\r\nMIME-Version: 1.0\r\nContent-Transfer-Encoding: quoted-printable\r\n\r\nTest=E0 and one\r\n";
String iso885915qpheader = "------ my footer =E0/=A4 ------";
String footer = "------ my footer \u00E0/\u20AC ------";
String res = processAddFooter(asciisource, footer);
assertEquals(asciisource + iso885915qpheader, res);
}
{code}
the result (using geronimo javamail) have an =80 instead of =A4 in the footer, meaning it probably used the cp1252 charset instead of the iso-8859-15 declared in the message header.
processAddFooter(asciisource, footer) create a JAMES Mail object from sources and then runs it through the AddFooter mailet.
The test has 4-5 dependencies and probably is not too hard to be run outside james:
http://svn.apache.org/repos/asf/james/server/trunk/phoenix-deployment/src/test/org/apache/james/transport/mailets/AddFooterTest.java
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (GERONIMO-4136) Encoding coversion issues with
geronimo javamail
Posted by "Rick McGuire (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/GERONIMO-4136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Rick McGuire resolved GERONIMO-4136.
------------------------------------
Resolution: Fixed
Committed revision 669901. Geronimo specs
Committed revision 669902. Geronimo javamail providers
The JAF content handler for text used by javamail contained a TODO comment that encoding support still needed to be implemented. Indeed it did. That was the cause the problem. All character/byte conversions in the data handler were being performed using the default encoding.
> Encoding coversion issues with geronimo javamail
> ------------------------------------------------
>
> Key: GERONIMO-4136
> URL: https://issues.apache.org/jira/browse/GERONIMO-4136
> Project: Geronimo
> Issue Type: Bug
> Security Level: public(Regular issues)
> Components: mail
> Affects Versions: 2.1.1
> Reporter: Stefano Bagnara
> Assignee: Rick McGuire
> Attachments: TestAddFooter.java
>
>
> We have a test trying to add a footer to a message using iso-8859-15 charset:
> {code:title=AddFooterTest.java}
> public void testAddFooterTextPlain() throws MessagingException, IOException {
> // quoted printable mimemessage text/plain
> String asciisource = "Subject: test\r\nContent-Type: text/plain; charset=ISO-8859-15\r\nMIME-Version: 1.0\r\nContent-Transfer-Encoding: quoted-printable\r\n\r\nTest=E0 and one\r\n";
> String iso885915qpheader = "------ my footer =E0/=A4 ------";
> String footer = "------ my footer \u00E0/\u20AC ------";
> String res = processAddFooter(asciisource, footer);
> assertEquals(asciisource + iso885915qpheader, res);
> }
> {code}
> the result (using geronimo javamail) have an =80 instead of =A4 in the footer, meaning it probably used the cp1252 charset instead of the iso-8859-15 declared in the message header.
> processAddFooter(asciisource, footer) create a JAMES Mail object from sources and then runs it through the AddFooter mailet.
> The test has 4-5 dependencies and probably is not too hard to be run outside james:
> http://svn.apache.org/repos/asf/james/server/trunk/phoenix-deployment/src/test/org/apache/james/transport/mailets/AddFooterTest.java
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (GERONIMO-4136) Encoding coversion issues with
geronimo javamail
Posted by "Rick McGuire (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/GERONIMO-4136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Rick McGuire updated GERONIMO-4136:
-----------------------------------
Attachment: TestAddFooter.java
Small test case to recreate this problem using only the javamail classes.
> Encoding coversion issues with geronimo javamail
> ------------------------------------------------
>
> Key: GERONIMO-4136
> URL: https://issues.apache.org/jira/browse/GERONIMO-4136
> Project: Geronimo
> Issue Type: Bug
> Security Level: public(Regular issues)
> Components: mail
> Affects Versions: 2.1.1
> Reporter: Stefano Bagnara
> Assignee: Rick McGuire
> Attachments: TestAddFooter.java
>
>
> We have a test trying to add a footer to a message using iso-8859-15 charset:
> {code:title=AddFooterTest.java}
> public void testAddFooterTextPlain() throws MessagingException, IOException {
> // quoted printable mimemessage text/plain
> String asciisource = "Subject: test\r\nContent-Type: text/plain; charset=ISO-8859-15\r\nMIME-Version: 1.0\r\nContent-Transfer-Encoding: quoted-printable\r\n\r\nTest=E0 and one\r\n";
> String iso885915qpheader = "------ my footer =E0/=A4 ------";
> String footer = "------ my footer \u00E0/\u20AC ------";
> String res = processAddFooter(asciisource, footer);
> assertEquals(asciisource + iso885915qpheader, res);
> }
> {code}
> the result (using geronimo javamail) have an =80 instead of =A4 in the footer, meaning it probably used the cp1252 charset instead of the iso-8859-15 declared in the message header.
> processAddFooter(asciisource, footer) create a JAMES Mail object from sources and then runs it through the AddFooter mailet.
> The test has 4-5 dependencies and probably is not too hard to be run outside james:
> http://svn.apache.org/repos/asf/james/server/trunk/phoenix-deployment/src/test/org/apache/james/transport/mailets/AddFooterTest.java
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (GERONIMO-4136) Encoding coversion issues with
geronimo javamail
Posted by "Rick McGuire (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/GERONIMO-4136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12606711#action_12606711 ]
Rick McGuire commented on GERONIMO-4136:
----------------------------------------
Sorry, hit the wrong key and sent the comment off prematurely.
I have the code at https://svn.apache.org/repos/asf/james/server/trunk checkout out. Is there another branch of the tree I'll be needing for some of these?
> Encoding coversion issues with geronimo javamail
> ------------------------------------------------
>
> Key: GERONIMO-4136
> URL: https://issues.apache.org/jira/browse/GERONIMO-4136
> Project: Geronimo
> Issue Type: Bug
> Security Level: public(Regular issues)
> Components: mail
> Affects Versions: 2.1.1
> Reporter: Stefano Bagnara
> Assignee: Rick McGuire
>
> We have a test trying to add a footer to a message using iso-8859-15 charset:
> {code:title=AddFooterTest.java}
> public void testAddFooterTextPlain() throws MessagingException, IOException {
> // quoted printable mimemessage text/plain
> String asciisource = "Subject: test\r\nContent-Type: text/plain; charset=ISO-8859-15\r\nMIME-Version: 1.0\r\nContent-Transfer-Encoding: quoted-printable\r\n\r\nTest=E0 and one\r\n";
> String iso885915qpheader = "------ my footer =E0/=A4 ------";
> String footer = "------ my footer \u00E0/\u20AC ------";
> String res = processAddFooter(asciisource, footer);
> assertEquals(asciisource + iso885915qpheader, res);
> }
> {code}
> the result (using geronimo javamail) have an =80 instead of =A4 in the footer, meaning it probably used the cp1252 charset instead of the iso-8859-15 declared in the message header.
> processAddFooter(asciisource, footer) create a JAMES Mail object from sources and then runs it through the AddFooter mailet.
> The test has 4-5 dependencies and probably is not too hard to be run outside james:
> http://svn.apache.org/repos/asf/james/server/trunk/phoenix-deployment/src/test/org/apache/james/transport/mailets/AddFooterTest.java
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (GERONIMO-4136) Encoding coversion issues with
geronimo javamail
Posted by "Rick McGuire (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/GERONIMO-4136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12606709#action_12606709 ]
Rick McGuire commented on GERONIMO-4136:
----------------------------------------
I'm attempting to reduce this to a simple test case for debugging, which looks doable. The one piece I seem to be missing is the AddFooter class. Where is that?
> Encoding coversion issues with geronimo javamail
> ------------------------------------------------
>
> Key: GERONIMO-4136
> URL: https://issues.apache.org/jira/browse/GERONIMO-4136
> Project: Geronimo
> Issue Type: Bug
> Security Level: public(Regular issues)
> Components: mail
> Affects Versions: 2.1.1
> Reporter: Stefano Bagnara
> Assignee: Rick McGuire
>
> We have a test trying to add a footer to a message using iso-8859-15 charset:
> {code:title=AddFooterTest.java}
> public void testAddFooterTextPlain() throws MessagingException, IOException {
> // quoted printable mimemessage text/plain
> String asciisource = "Subject: test\r\nContent-Type: text/plain; charset=ISO-8859-15\r\nMIME-Version: 1.0\r\nContent-Transfer-Encoding: quoted-printable\r\n\r\nTest=E0 and one\r\n";
> String iso885915qpheader = "------ my footer =E0/=A4 ------";
> String footer = "------ my footer \u00E0/\u20AC ------";
> String res = processAddFooter(asciisource, footer);
> assertEquals(asciisource + iso885915qpheader, res);
> }
> {code}
> the result (using geronimo javamail) have an =80 instead of =A4 in the footer, meaning it probably used the cp1252 charset instead of the iso-8859-15 declared in the message header.
> processAddFooter(asciisource, footer) create a JAMES Mail object from sources and then runs it through the AddFooter mailet.
> The test has 4-5 dependencies and probably is not too hard to be run outside james:
> http://svn.apache.org/repos/asf/james/server/trunk/phoenix-deployment/src/test/org/apache/james/transport/mailets/AddFooterTest.java
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.