[CAMEL-6045] Camel Email Component Missing Attachments

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.6.0
• Fix Version/s: 2.9.6, 2.10.4, 2.11.0
• Component/s: camel-mail
• Patch Info: Patch Available
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: Christoph Giera
• Created: 07/Feb/13 14:00
• Updated: 23/Oct/23 07:19
• Resolved: 07/Feb/13 14:53

Description

see http://camel.465427.n5.nabble.com/Camel-Email-Component-Missing-Attachments-td3386382.html#a5727102

The disposition field is optional(see RFC 2183), so it is possible that camel misses attachments.

                if (disposition != null && (disposition.equalsIgnoreCase(Part.ATTACHMENT) || disposition.equalsIgnoreCase(Part.INLINE))) {
                    // only add named attachments
                    String fileName = part.getFileName();
                    if (fileName != null) {
                        LOG.debug("Mail contains file attachment: " + fileName);
                        // Parts marked with a disposition of Part.ATTACHMENT are clearly attachments
                        CollectionHelper.appendValue(map, fileName, part.getDataHandler());
                    }
                }

Adding the fileName check to the if should resolve the issue.

Issue Links

• relates to: CAMEL-20028 camel-mail - Missing attachments if disposition not set

Sub-Tasks

• Attachments with null filename should not be allowed: Resolved