You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by "amirhadadi (via GitHub)" <gi...@apache.org> on 2023/02/10 09:47:40 UTC

[GitHub] [logging-log4j2] amirhadadi opened a new issue, #1274: RewriteAppender ignores the location requirement of its appenders

amirhadadi opened a new issue, #1274:
URL: https://github.com/apache/logging-log4j2/issues/1274

   Currently `RewriteAppender` does not override `requiresLocation` from `AbstractAppender` making `RewriteAppender::requiresLocation` always return `false`. This results in the location not being passed to its underlying appenders when needed.
   Instead, `RewriteAppender::requiresLocation` should return true when any of its appenders requires a location.
   


-- 
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.apache.org

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


[GitHub] [logging-log4j2] amirhadadi commented on issue #1274: RewriteAppender ignores the location requirement of its appenders

Posted by "amirhadadi (via GitHub)" <gi...@apache.org>.
amirhadadi commented on issue #1274:
URL: https://github.com/apache/logging-log4j2/issues/1274#issuecomment-1436565448

   @ppkarwasz that version fixes the issue. Thanks for the quick fix!


-- 
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


[GitHub] [logging-log4j2] ppkarwasz closed issue #1274: RewriteAppender ignores the location requirement of its appenders

Posted by "ppkarwasz (via GitHub)" <gi...@apache.org>.
ppkarwasz closed issue #1274: RewriteAppender ignores the location requirement of its appenders
URL: https://github.com/apache/logging-log4j2/issues/1274


-- 
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


[GitHub] [logging-log4j2] ppkarwasz commented on issue #1274: RewriteAppender ignores the location requirement of its appenders

Posted by "ppkarwasz (via GitHub)" <gi...@apache.org>.
ppkarwasz commented on issue #1274:
URL: https://github.com/apache/logging-log4j2/issues/1274#issuecomment-1428675976

   @amirhadadi,
   
   Thank you for noticing, it will be fixed in the next release. Can you check if the latest `2.19.1-SNAPSHOT` available on https://repository.apache.org/snapshots fixes your problem?


-- 
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