You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by bu...@apache.org on 2009/05/03 01:04:50 UTC

DO NOT REPLY [Bug 47142] New: add buildEmailBody() method to log4j SMTPAppender

https://issues.apache.org/bugzilla/show_bug.cgi?id=47142

           Summary: add buildEmailBody() method to log4j SMTPAppender
           Product: Log4j
           Version: 1.2
          Platform: PC
        OS/Version: Mac OS X 10.4
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Appender
        AssignedTo: log4j-dev@logging.apache.org
        ReportedBy: apache-bugzilla@seansullivan.com


Patch attached.

This patch adds a new method:  protected StringBuffer buildEmailBody()

This method makes it easier for subclasses to customize the email message body.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org

DO NOT REPLY [Bug 47142] add buildEmailBody() method to log4j SMTPAppender

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47142





--- Comment #2 from Curt Arnold <ca...@apache.org>  2009-05-11 21:12:43 PST ---
Having a method return StringBuffer was a little strange.  I modified the name
to String formatBody().  It will cause an extra StringBuffer allocation if
encoding fails, but that is an edge case.

Committed in rev 773775.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org

DO NOT REPLY [Bug 47142] add buildEmailBody() method to log4j SMTPAppender

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47142


Curt Arnold <ca...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org

DO NOT REPLY [Bug 47142] add buildEmailBody() method to log4j SMTPAppender

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47142





--- Comment #1 from Sean C. Sullivan <ap...@seansullivan.com>  2009-05-02 16:06:31 PST ---
Created an attachment (id=23598)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23598)
patch for SMTPAppender.java

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org