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 "Remko Popma (JIRA)" <ji...@apache.org> on 2014/09/03 07:30:52 UTC

[jira] [Commented] (LOG4J2-811) Log4j2's SimpleLogger Implementation Issue for var args method's

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

Remko Popma commented on LOG4J2-811:
------------------------------------

Hm... If we check and we find that the application made a mistake and provided fewer args than the pattern requires, what should log4j do? I think we should probably throw an exception, and IndexOutOfBounds is a pretty good choice, I'd say. What do you propose?

> Log4j2's SimpleLogger Implementation Issue for var args method's
> ----------------------------------------------------------------
>
>                 Key: LOG4J2-811
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-811
>             Project: Log4j 2
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 2.0.1
>            Reporter: Yogesh Rao
>              Labels: patch
>
> There seems to be an issue with SimpleLogger implementation provided by log4j2. The issue seems to be in the new improved API supporting placeholders and var args when called with an Object Array of size 0.
> for e.g logger.error("Hello World {} in {} " , new Object[0]);
> A statement above results in an error as shown below
> ERROR StatusLogger Unable to locate a logging implementation, using SimpleLogger
> Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: -1
>        at org.apache.logging.log4j.simple.SimpleLogger.logMessage(SimpleLogger.java:157)
>        at org.apache.logging.log4j.spi.AbstractLogger.logMessage(AbstractLogger.java:1347)
>        at org.apache.logging.log4j.spi.AbstractLogger.logIfEnabled(AbstractLogger.java:1312)
>        at org.apache.logging.log4j.spi.AbstractLogger.error(AbstractLogger.java:539)
>        at TestError.main(TestError.java:21)
> Solution to place a check in SimpleLogger for checking the size of the array . 



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

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