CAMEL-7873 Add capability to sort incoming mail messages #277
Conversation
camann9
changed the title
|
|
||
| import com.sun.mail.imap.IMAPFolder; | ||
| import com.sun.mail.imap.IMAPStore; | ||
| import com.sun.mail.imap.SortTerm; | ||
| import org.apache.camel.Exchange; |
There was a problem hiding this comment.
We are not supposed to use com.sun packages unless you have good explain of it.
There was a problem hiding this comment.
But then we can not find out if a folder is an IMAP folder and we can not call the specific methods we need for sorting (only IMAP supports sorting).
The dependencies I referenced are all part of the maven dependency javax.mail/mail, they are not dependent on the type of JDK/JRE we are running (even though they have the com.sun prefix).
To make it short: there is no way of implementing this functionality without using stuff from com.sun.mail. But I think in this situation it not a problem.
There was a problem hiding this comment.
OK, I got it, we can treat the com.sun.mail as a third part jar. I will merged the code shortly.
|
Has been merged, closing it |
Camel should have the ability to sort incoming mail messages. This patch adds that ability. There is one caveat:
POP3 does not support sorting and for IMAP it's an extension. To remediate this problem this patch can either use On-Server sorting where the capability exists and Client side sorting for servers that do not support sorting.
There is a test case that tests the route integration and multiple test cases that test the sorting component.
I honestly don't know how to add this to the documentation.