MailMessageSender.java eats exception chain - which does not allow for proper 
dubuging and logging
--------------------------------------------------------------------------------------------------

         Key: COCOON-1869
         URL: http://issues.apache.org/jira/browse/COCOON-1869
     Project: Cocoon
        Type: Bug

  Components: Blocks: Mail  
    Versions: 2.1.9    
    Reporter: Kaj Kandler


line 448 eats the chain of exceptions, which is bad for finding bugs and 
misconfiguration. 

It should better read:

        } catch (MessagingException me) {
            throw me;

or 

        } catch (MessagingException me) {
            throw me; // new MessagingException(me.getMessage(),me);

or be eliminated all together, following the principle, if you can't do 
anything about an exception and don't need to aggregate it, because of API 
design, don't catch it in the first place!




-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to