AddFooter mailet javax.mail.internet.ParseException

• Key: JAMES-456
• Type: Bug
• Status: Closed
• Resolution: Fixed
• Priority: Major
• Assignee: Stefano Bagnara
• Reporter: Norman Maurer
• Component/s: Matchers/Mailets (bundled)
• Fix Version/s: 2.3.0

Description

I just found a javax.mail.internet.ParseException in my logs. Im not 100% sure that this is a bug:

15/03/06 10:44:24 ERROR spoolmanager: Exception in processor <excubator>
javax.mail.internet.ParseException
        at javax.mail.internet.ParameterList.<init>(ParameterList.java:81)
        at javax.mail.internet.ContentType.<init>(ContentType.java:82)
        at javax.mail.internet.MimeBodyPart.updateHeaders(MimeBodyPart.java:1105)
        at javax.mail.internet.MimeBodyPart.updateHeaders(MimeBodyPart.java:863)
        at javax.mail.internet.MimeMultipart.updateHeaders(MimeMultipart.java:23
9)
        at javax.mail.internet.MimeBodyPart.updateHeaders(MimeBodyPart.java:1111)
        at javax.mail.internet.MimeMessage.updateHeaders(MimeMessage.java:1927)
        at javax.mail.internet.MimeMessage.saveChanges(MimeMessage.java:1908)
        at org.apache.james.core.MimeMessageWrapper.saveChanges(MimeMessageWrapper.java:883)
        at org.apache.james.core.MimeMessageCopyOnWriteProxy.saveChanges(MimeMessageCopyOnWriteProxy.java:711)
        at org.apache.james.transport.mailets.AddFooter.service(AddFooter.java:66)
        at de.byteaction.excubator.james.transport.mailets.AddFooterFromAttribute.service(AddFooterFromAttribute.java:59)
        at org.apache.james.transport.LinearProcessor.service(LinearProcessor.java:414)
        at org.apache.james.transport.JamesSpoolManager.process(JamesSpoolManager.java:397)
        at org.apache.james.transport.JamesSpoolManager.run(JamesSpoolManager.java:306)
        at java.lang.Thread.run(Thread.java:595)

Comments

[Stefano Bagnara]

Can you find a mail to reproduce the issue?

[Norman Maurer]

not yet.. I will attach one if i see one again in the logs and deadletter table.

[Norman Maurer]

Here is such a message that cause this exception..

I just did a : select * from deadletter where error_message IS NULL order by last_updated desc limit 1 into outfile '/tmp/exception.dump;

[Norman Maurer]

I just did this:
select * from deadletter where error_message IS NULL order by last_updated desc limit 1 into outfile '/tmp/exception.dump';

in the file is the output.

[Stefano Bagnara]

The message you attached has the body of a multipart message but has no content-type.
Your AddFooter is probably different from mine because line numbers in your stacktrace don't match.
Can you reproduce the problem in a test?
Look at the AddFooterTest.java source code to see how we run unit tests for AddFooter.

[Norman Maurer]

I don't see this behavior for ages now... I close it.