camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CAMEL-7254) MailComponent 'peek' doesn't work due to programming error
Date Fri, 28 Feb 2014 13:11:19 GMT

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

ASF GitHub Bot commented on CAMEL-7254:
---------------------------------------

Github user davidkarlsen closed the pull request at:

    https://github.com/apache/camel/pull/103


> MailComponent 'peek' doesn't work due to programming error
> ----------------------------------------------------------
>
>                 Key: CAMEL-7254
>                 URL: https://issues.apache.org/jira/browse/CAMEL-7254
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 2.12.2
>            Reporter: Matt Nathan
>            Assignee: Willem Jiang
>            Priority: Critical
>             Fix For: 2.12.4, 2.13.0, 2.11.5
>
>
> The MailComponent peek setting doesn't work due to the following line:
> {code:java}
>     private void peekMessage(Message mail) {
>         // this only applies to IMAP messages which has a setPeek method
>         if (mail.getClass().getName().startsWith("IMAP")) {
>             try {
>                 LOG.trace("Calling setPeek(true) on mail message {}", mail);
>                 IntrospectionSupport.setProperty(mail, "peek", true);
>             } catch (Throwable e) {
>                 // ignore
>                 LOG.trace("Error setting peak property to true on: " + mail + ". This
exception is ignored.", e);
>             }
>         }
>     }
> {code}
> The line that checks the class name for IMAP should be using getSimpleName. Otherwise
it's checking the package name instead.
> This effectively means that rollback of processing in camel mail is not supported.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Mime
View raw message