[jira] Updated: (GERONIMO-4033) QuotedPrintableEncoder.decode(InputStream) doesn't handle space before soft newline

2008-05-19 Thread aaron pieper (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-4033?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

aaron pieper updated GERONIMO-4033:
---

Summary: QuotedPrintableEncoder.decode(InputStream) doesn't handle space 
before soft newline  (was: QuotedPrintableEncoder.decore(InputStream) doesn't 
handle space before soft newline)

> QuotedPrintableEncoder.decode(InputStream) doesn't handle space before soft 
> newline
> ---
>
> Key: GERONIMO-4033
> URL: https://issues.apache.org/jira/browse/GERONIMO-4033
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: mail
>Affects Versions: 1.x
>Reporter: aaron pieper
> Attachments: patch.txt
>
>
> QuotedPrintableEncoder.decode(InputStream) does not correctly handle the case 
> where a space occurs before a soft newline. It decodes the space correctly, 
> but the next character decodes as "=".
> In other words, where "A =\r\nB" should decode as "A B", it instead decodes 
> as "A =\r\nB"
> This is a problem in geronimo-javamail_1.4_spec-1.3. I am reporting this as 
> version 1.x, as this is the closest match.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (GERONIMO-4033) QuotedPrintableEncoder.decore(InputStream) doesn't handle space before soft newline

2008-05-19 Thread aaron pieper (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-4033?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

aaron pieper updated GERONIMO-4033:
---

Attachment: patch.txt

Proposed patch

> QuotedPrintableEncoder.decore(InputStream) doesn't handle space before soft 
> newline
> ---
>
> Key: GERONIMO-4033
> URL: https://issues.apache.org/jira/browse/GERONIMO-4033
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: mail
>Affects Versions: 1.x
>Reporter: aaron pieper
> Attachments: patch.txt
>
>
> QuotedPrintableEncoder.decode(InputStream) does not correctly handle the case 
> where a space occurs before a soft newline. It decodes the space correctly, 
> but the next character decodes as "=".
> In other words, where "A =\r\nB" should decode as "A B", it instead decodes 
> as "A =\r\nB"
> This is a problem in geronimo-javamail_1.4_spec-1.3. I am reporting this as 
> version 1.x, as this is the closest match.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Created: (GERONIMO-4033) QuotedPrintableEncoder.decore(InputStream) doesn't handle space before soft newline

2008-05-19 Thread aaron pieper (JIRA)
QuotedPrintableEncoder.decore(InputStream) doesn't handle space before soft 
newline
---

 Key: GERONIMO-4033
 URL: https://issues.apache.org/jira/browse/GERONIMO-4033
 Project: Geronimo
  Issue Type: Bug
  Security Level: public (Regular issues)
  Components: mail
Affects Versions: 1.x
Reporter: aaron pieper


QuotedPrintableEncoder.decode(InputStream) does not correctly handle the case 
where a space occurs before a soft newline. It decodes the space correctly, but 
the next character decodes as "=".

In other words, where "A =\r\nB" should decode as "A B", it instead decodes as 
"A =\r\nB"

This is a problem in geronimo-javamail_1.4_spec-1.3. I am reporting this as 
version 1.x, as this is the closest match.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (GERONIMO-4022) QuotedPrintableDecoderStream read should return -1 when no more data

2008-05-16 Thread aaron pieper (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-4022?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

aaron pieper updated GERONIMO-4022:
---

Attachment: patch.txt

> QuotedPrintableDecoderStream read should return -1 when no more data
> 
>
> Key: GERONIMO-4022
> URL: https://issues.apache.org/jira/browse/GERONIMO-4022
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: mail
>Affects Versions: 1.x
>Reporter: aaron pieper
>Priority: Minor
> Attachments: patch.txt
>
>
> QuotedPrintableDecoderStream.read(byte[], int, int) returns 0 when there are 
> no more bytes to be read. It should return -1.
> One result of this errant return value is that 
> org.apache.commons.io.IOUtils.copy(InputStream, OutputStream) will enter an 
> endless loop if given a QuotedPrintableDecoderStream.
> Note: This affects geronimo-javamail_1.4_spec-1.3.jar. I'm not sure what 
> version of geronimo this corresponds to, so I'm recording it as 1.x which is 
> the closest match

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Created: (GERONIMO-4022) QuotedPrintableDecoderStream read should return -1 when no more data

2008-05-16 Thread aaron pieper (JIRA)
QuotedPrintableDecoderStream read should return -1 when no more data


 Key: GERONIMO-4022
 URL: https://issues.apache.org/jira/browse/GERONIMO-4022
 Project: Geronimo
  Issue Type: Bug
  Security Level: public (Regular issues)
  Components: mail
Affects Versions: 1.x
Reporter: aaron pieper


QuotedPrintableDecoderStream.read(byte[], int, int) returns 0 when there are no 
more bytes to be read. It should return -1.

One result of this errant return value is that 
org.apache.commons.io.IOUtils.copy(InputStream, OutputStream) will enter an 
endless loop if given a QuotedPrintableDecoderStream.

Note: This affects geronimo-javamail_1.4_spec-1.3.jar. I'm not sure what 
version of geronimo this corresponds to, so I'm recording it as 1.x which is 
the closest match

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (GERONIMO-4022) QuotedPrintableDecoderStream read should return -1 when no more data

2008-05-16 Thread aaron pieper (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-4022?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

aaron pieper updated GERONIMO-4022:
---

Priority: Minor  (was: Major)

> QuotedPrintableDecoderStream read should return -1 when no more data
> 
>
> Key: GERONIMO-4022
> URL: https://issues.apache.org/jira/browse/GERONIMO-4022
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: mail
>Affects Versions: 1.x
>Reporter: aaron pieper
>Priority: Minor
>
> QuotedPrintableDecoderStream.read(byte[], int, int) returns 0 when there are 
> no more bytes to be read. It should return -1.
> One result of this errant return value is that 
> org.apache.commons.io.IOUtils.copy(InputStream, OutputStream) will enter an 
> endless loop if given a QuotedPrintableDecoderStream.
> Note: This affects geronimo-javamail_1.4_spec-1.3.jar. I'm not sure what 
> version of geronimo this corresponds to, so I'm recording it as 1.x which is 
> the closest match

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.