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

Scott Severtson commented on LOG4J2-131:
----------------------------------------

1. I don't believe that any of the changes from LOG4J2-136 are needed - it's 
entirely new code.
2. I pushed the JavaMail .jar files everywhere, as the tests in those projects 
were attempting to load all Appenders, including the new SMTPAppender. Without 
the JavaMail jars, multiple failures occurred. If there's a better solution 
(i.e. changing the subproject tests to not load SMTPAppender, please let me 
know.
3. I agree - SMTPSessionManager *could* do more work, but I wouldn't put the 
CyclicBuffer in there. Consistent with Log4J 1.x, the buffer is 
consumed/cleared when sending an event, and if instances have different 
Filters, they could unfortunately interfere with each other.

Thanks, and let me know if there's anything I can do to help.
                
> Create SMTPAppender
> -------------------
>
>                 Key: LOG4J2-131
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-131
>             Project: Log4j 2
>          Issue Type: New Feature
>          Components: Appenders
>            Reporter: Christian Grobmeier
>         Attachments: SMTPAppender.patch
>
>
> Somebody in twitterverse reverted back to log4j 1.2 because he missed the 
> SMTP Appender

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to