[
https://issues.apache.org/jira/browse/MIME4J-60?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12652013#action_12652013
]
Oleg Kalnichevski commented on MIME4J-60:
-----------------------------------------
@Stefano
The issue of consistency
=========================
1) RootInputStream
RootInputStream is broken for MIME streams with binary transfer coding and
should not have existed in the first place, but I will happily make it
consistent with the rest of the code.
2) BufferedLinedReaderInputStream:
New line = CRLF | LF
3) AbstractEntity
New line = CRLF | LF
4) QuotedPrintableInputStream
Cannot comment. Will happily investigate.
5) MimeUtil#getHeaderParams
Cannot comment. Will happily investigate.
6) MimeBoundaryInputStream
New line = CRLF | LF
Out of 6 cases I personally think there are only three legitimate cases that
need attention.
The issue of isolated CR
=========================
I consider isolated CR a non-issue, but fully admit to my HTTP bias. However, I
am prepared to implement two modes of operation for all relevant classes as
outlined in my previous post.
I can commit myself to taking the following actions:
option 1) Fixing RootInputStream, investigating QuotedPrintableInputStream and
MimeUtil#getHeaderParams and dropping isolated CR as a non-issue.
option 2) Fixing RootInputStream, QuotedPrintableInputStream,
MimeUtil#getHeaderParams and implementing strict / lenient parsing modes for
all relevant classes.
If you are not happy with either of the options, I once again will have to step
away and let someone else take the lead
I clearly prefer option 1
@Markus
Besides obvious performance implications, simple input streaming wrapping and
filtering will result in corruption of parts with binary transfer coding (used
very frequently when transferring MIME coded content over HTTP protocol)
> Configurable strategy for line delimiters
> -----------------------------------------
>
> Key: MIME4J-60
> URL: https://issues.apache.org/jira/browse/MIME4J-60
> Project: JAMES Mime4j
> Issue Type: Wish
> Affects Versions: 0.4
> Reporter: Stefano Bagnara
> Fix For: 0.6
>
> Attachments: MIME4J-60-readLine-returns-no-newline.patch,
> newlines-tests.zip, newlinestrat.patch
>
>
> There is an ongoing discussion about how we should deal with non canonical
> line endings (isolated LF and/or isolated CR).
> This issue is to track discussion results and proposed patches.
--
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]