You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by ychawla <pr...@yahoo.com> on 2011/12/22 18:58:20 UTC

Re: Camel Email Component Missing Attachments

Hi Phil,
I saw the same behavior for a Mail Server we are going against.  I am
thinking it would be great to have a configuration on the endpoint like
'useDispositionForAttachmentRetrieval'.  

There can a check around this block:

  if (disposition != null && (disposition.equalsIgnoreCase(Part.ATTACHMENT)
|| disposition.equalsIgnoreCase(Part.INLINE))) { }

The critical check on the file name should still be performed:

                    String fileName = part.getFileName();
                    if (fileName != null) 

I would be willing to work on a patch if the Camel commiters think this is a
good approach.

Thanks,
Yogesh

--
View this message in context: http://camel.465427.n5.nabble.com/Camel-Email-Component-Missing-Attachments-tp3386382p5095264.html
Sent from the Camel Development mailing list archive at Nabble.com.

Re: Camel Email Component Missing Attachments

Posted by cgiera <ch...@mic-cust.com>.
Hello,

yes this works for me. I have created an issue and provided a patch. see
https://issues.apache.org/jira/browse/CAMEL-6045

Please let me know if I have done something wrong or could make something
better.

kind regards,
Christoph



--
View this message in context: http://camel.465427.n5.nabble.com/Camel-Email-Component-Missing-Attachments-tp3386382p5727131.html
Sent from the Camel Development mailing list archive at Nabble.com.

Re: Camel Email Component Missing Attachments

Posted by Claus Ibsen <cl...@gmail.com>.
On Thu, Feb 7, 2013 at 12:09 PM, cgiera <ch...@mic-cust.com> wrote:
> Hello,
>
> I know this is an old post, but I'm facing the same problem.
>
> Could this be solved by simply adding a fileName check to the if statement?
>

I suggest to try patch the code and test it.
And if it helps, then we love contributions
http://camel.apache.org/contributing.html

>
> kind regards,
> Christoph
>
>
>
>
> --
> View this message in context: http://camel.465427.n5.nabble.com/Camel-Email-Component-Missing-Attachments-tp3386382p5727102.html
> Sent from the Camel Development mailing list archive at Nabble.com.



-- 
Claus Ibsen
-----------------
Red Hat, Inc.
FuseSource is now part of Red Hat
Email: cibsen@redhat.com
Web: http://fusesource.com
Twitter: davsclaus
Blog: http://davsclaus.com
Author of Camel in Action: http://www.manning.com/ibsen

Re: Camel Email Component Missing Attachments

Posted by cgiera <ch...@mic-cust.com>.
Hello,

I know this is an old post, but I'm facing the same problem. 

Could this be solved by simply adding a fileName check to the if statement?


kind regards,
Christoph




--
View this message in context: http://camel.465427.n5.nabble.com/Camel-Email-Component-Missing-Attachments-tp3386382p5727102.html
Sent from the Camel Development mailing list archive at Nabble.com.

Re: Camel Email Component Missing Attachments

Posted by Claus Ibsen <cl...@gmail.com>.
Hi

Yes we love contributions. Please create a JIRA and provide a patch.
http://camel.apache.org/contributing.html


On Thu, Dec 22, 2011 at 6:58 PM, ychawla <pr...@yahoo.com> wrote:
> Hi Phil,
> I saw the same behavior for a Mail Server we are going against.  I am
> thinking it would be great to have a configuration on the endpoint like
> 'useDispositionForAttachmentRetrieval'.
>
> There can a check around this block:
>
>  if (disposition != null && (disposition.equalsIgnoreCase(Part.ATTACHMENT)
> || disposition.equalsIgnoreCase(Part.INLINE))) { }
>
> The critical check on the file name should still be performed:
>
>                    String fileName = part.getFileName();
>                    if (fileName != null)
>
> I would be willing to work on a patch if the Camel commiters think this is a
> good approach.
>
> Thanks,
> Yogesh
>
> --
> View this message in context: http://camel.465427.n5.nabble.com/Camel-Email-Component-Missing-Attachments-tp3386382p5095264.html
> Sent from the Camel Development mailing list archive at Nabble.com.



-- 
Claus Ibsen
-----------------
FuseSource
Email: cibsen@fusesource.com
Web: http://fusesource.com
Twitter: davsclaus, fusenews
Blog: http://davsclaus.blogspot.com/
Author of Camel in Action: http://www.manning.com/ibsen/