[jira] [Commented] (TIKA-2044) MboxParser wrongly concatenates multiple text lines into single header line

["Nick Burch (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TIKA-2044?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15396477#comment-15396477 ] 

Nick Burch commented on TIKA-2044:
----------------------------------

Are you able to reproduce this in a simple junit unit test case? Perhaps using one of our existing mbox files, or otherwise with a nice simple one of your own? If you could, that'd be a great help, both for ensuring the fix is in + keeping it fixed in the future!

> MboxParser wrongly concatenates multiple text lines into single header line
> ---------------------------------------------------------------------------
>
>                 Key: TIKA-2044
>                 URL: https://issues.apache.org/jira/browse/TIKA-2044
>             Project: Tika
>          Issue Type: Bug
>    Affects Versions: 1.13
>         Environment: Tika 1.13, and 1.14 nightly build at the time of this writing
>            Reporter: Vjeran Marcinko
>
> MboxParser combines multiple text lines into single header value by (suposedly) using LIFO structure (stack - java deque), but instead it uses FIFO (queue) to fetch last inserted line and to extend it with current line in incorrect way:
> Current code:
> Queue<String> multiline = new LinkedList<String>();
> ... few lines below...
> String latestLine = multiline.poll();
> Whereas it should be:
> Deque<String> multiline = new LinkedList<String>();
> ... few lines below...
> String latestLine = multiline.pollLast();



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)