You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by GitBox <gi...@apache.org> on 2021/12/16 16:34:39 UTC

[GitHub] [logging-log4j2] garydgregory commented on a change in pull request #630: Log4j2 is still vulnerable and underspecified. This updates documenta…

garydgregory commented on a change in pull request #630:
URL: https://github.com/apache/logging-log4j2/pull/630#discussion_r770718080



##########
File path: log4j-api/src/main/java/org/apache/logging/log4j/Logger.java
##########
@@ -137,7 +137,7 @@
      * Logs a message CharSequence with the {@link Level#DEBUG DEBUG} level.
      *
      * @param marker the marker data specific to this log statement
-     * @param message the message CharSequence to log.
+     * @param message the message CharSequence to log. THIS MAY OR MAY NOT BE INTERPRETED AS A MESSAGE OBJECT

Review comment:
       -1 on the UPPER CASE SHOUTING. In addition, the text is not really helpful, it does not tell you why this might matter.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org