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

Gary Gregory commented on LOG4J2-1099:
--------------------------------------

Weclome back!

> AbstractStringLayout implements Serializable, but is not Serializable
> ---------------------------------------------------------------------
>
>                 Key: LOG4J2-1099
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-1099
>             Project: Log4j 2
>          Issue Type: Bug
>          Components: Layouts
>    Affects Versions: 2.3
>            Reporter: Louis Burton
>            Assignee: Matt Sicker
>            Priority: Critical
>
> {{org.apache.logging.log4j.core.layout.AbstractLayout}} line 34 :
> {code}
>     // TODO: Charset is not serializable. Implement read/writeObject() ?
>     private final Charset charset;
> {code}
> The developer has recognised that this class claims to be serializable, but 
> is not actually serializable.
> This actually has wide impact due to the fact that the Logger is holding onto 
> the Layout via the 
> {{org.apache.logging.log4j.core.Logger.PrivateConfig#config}} (XML in my 
> case). Many projects, including Spring, do not use static Loggers and prefer 
> getClass type approaches off of their abstract classes, i.e.:
> {code}
> protected final Log logger = LogFactory.getLog(getClass());
> {code} 
> This actually can lead to use of spring session beans, which are serialized 
> with the session, trying to serialize the logger also and failing due to this 
> bug.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to