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

ASF GitHub Bot commented on JAMES-2263:
---------------------------------------

Github user chibenwa commented on a diff in the pull request:

    https://github.com/apache/james-project/pull/91#discussion_r157435051
  
    --- Diff: 
server/mailet/mailetcontainer-camel/src/test/java/org/apache/james/mailetcontainer/lib/AbstractStateMailetProcessorTest.java
 ---
    @@ -201,7 +202,7 @@ public void afterMailet(Mailet m, String mailName, 
String state, long processTim
             } catch (MessagingException e) {
                 catched = true;
             }
    -        assertTrue(catched);
    +        assertFalse(catched);
    --- End diff --
    
    ```
    boolean catched = false;
    try {
        // do something
    } catch(Exception e) {
        catched = true;
    }
    assertFalse(catched);
    ```
    
    Is strictly equivalent to : 
    
    ```
    // do something
    ```
    
    (and the second options prints error details, easing debugging...)


> onMailetException doesn't work even the default onMailetException="error"
> -------------------------------------------------------------------------
>
>                 Key: JAMES-2263
>                 URL: https://issues.apache.org/jira/browse/JAMES-2263
>             Project: James Server
>          Issue Type: Bug
>          Components: Matchers/Mailets (bundled), SpoolManager & Processors
>    Affects Versions: 3.0.0-beta5
>            Reporter: Taro App
>
> Tested with 3.0.1. When a mailet raises an exception, the message is not sent 
> to the processor specified with onMailetException. Only 
> onMailetException="ignore" works. Other values don't work, the even the 
> default onMailetException="error". Instead the message is somehow processed 7 
> times and then discarded.
> See org.apache.james.mailetcontainer.impl.camel.CamelProcessor.java:106, the 
> exception and the onMailetException value is passed to 
> ProcessorUtil.handleException, which then re-throw the exception. The 
> re-thrown exception is never catched and the mail object is discard at 
> org.apache.james.mailetcontainer.impl.JamesMailSpooler.java:183.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to