You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2019/05/13 05:38:00 UTC
[jira] [Commented] (LOG4J2-2561) In log4j-core AbstractStringLayout
java version detection is not JEP223 compatible
[ https://issues.apache.org/jira/browse/LOG4J2-2561?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16838279#comment-16838279 ]
ASF subversion and git services commented on LOG4J2-2561:
---------------------------------------------------------
Commit 045bb97f53fc886f91866d4a0a62160c71aea48e in logging-log4j2's branch refs/heads/release-2.x from Ralph Goers
[ https://gitbox.apache.org/repos/asf?p=logging-log4j2.git;h=045bb97 ]
LOG4J2-2561 - java version detection was not JEP223 compatible
> In log4j-core AbstractStringLayout java version detection is not JEP223 compatible
> ----------------------------------------------------------------------------------
>
> Key: LOG4J2-2561
> URL: https://issues.apache.org/jira/browse/LOG4J2-2561
> Project: Log4j 2
> Issue Type: Bug
> Components: Core
> Affects Versions: 2.11.2
> Reporter: Ulrich Enslin
> Priority: Major
> Labels: easyfix, pull-request-available
> Time Spent: 3.5h
> Remaining Estimate: 0h
>
> The function isPreJava8() will detect java 9 and higher as being pre java 8, because the logic does not take JEP223 into account.
>
> The same logic is correct in org.apache.logging.log4j.util.Constants#getMajorVersion, so simply modified the logic in AbstractStringLayout to work the same in the PR.
>
> This seems like a small change, but it prevents log4j from being JDK 9 and up compatible.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)