[ 
https://issues.apache.org/jira/browse/MIME4J-79?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12633918#action_12633918
 ] 

Markus Wiederkehr commented on MIME4J-79:
-----------------------------------------

I think this is a good idea and your patch looks fine to me.

But maybe the same concept should also be applied to 
QuotedPrintableOutputStream? Although it is an inner class of CodecUtil it can 
be accessed the same way as the base64 version through 
wrapQuotedPrintable(OutputStream, boolean). Maybe it should even become a 
public class for consistency reasons..

> Message.writeTo() prematurely closes output stream if transfer encoding is 
> BASE64
> ---------------------------------------------------------------------------------
>
>                 Key: MIME4J-79
>                 URL: https://issues.apache.org/jira/browse/MIME4J-79
>             Project: JAMES Mime4j
>          Issue Type: Bug
>    Affects Versions: 0.5
>            Reporter: Markus Wiederkehr
>            Priority: Blocker
>             Fix For: 0.5
>
>         Attachments: Base64BugDemo.java, mime4j-base64.patch, 
> mime4j-close-codec.patch
>
>
> Entity.writeTo(OutputStream out, int mode) wraps the output stream in a 
> Base64OutputStream if the transfer encoding is BASE64. Later the wrapper 
> stream gets closed and (despite of a comment that says otherwise) the inner 
> stream gets closed, too.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to