You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by andrei-ivanov <gi...@git.apache.org> on 2018/09/17 14:18:47 UTC
[GitHub] logging-log4j-audit pull request #8: [LOG4J2-2421]
GitHub user andrei-ivanov opened a pull request:
https://github.com/apache/logging-log4j-audit/pull/8
[LOG4J2-2421]
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/andrei-ivanov/logging-log4j-audit LOG4J2-2421
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/logging-log4j-audit/pull/8.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #8
----
commit af89554f0d34ebd8263b0693cd8c6a6095b6b772
Author: Andrei Ivanov <an...@...>
Date: 2018-09-17T14:11:02Z
[LOG4J2-2421] check for the presence of required ThreadContext atributes
commit baa2d74d9e4437629d02f6670d87c5430d6e5e69
Author: Andrei Ivanov <an...@...>
Date: 2018-09-17T14:14:11Z
[LOG4J2-2421] iterate only over the defined request context attributes as the ThreadContext can have others extra
commit 0c3a58a33923a4eabac20669a89b8cc7dbc21575
Author: Andrei Ivanov <an...@...>
Date: 2018-09-17T14:17:39Z
[LOG4J2-2421] remove pointless creation of array
----
---
[GitHub] logging-log4j-audit issue #8: [LOG4J2-2421]
Posted by andrei-ivanov <gi...@git.apache.org>.
Github user andrei-ivanov commented on the issue:
https://github.com/apache/logging-log4j-audit/pull/8
Should be fine now 🙂
---
[GitHub] logging-log4j-audit pull request #8: [LOG4J2-2421]
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/logging-log4j-audit/pull/8
---
[GitHub] logging-log4j-audit issue #8: [LOG4J2-2421]
Posted by rgoers <gi...@git.apache.org>.
Github user rgoers commented on the issue:
https://github.com/apache/logging-log4j-audit/pull/8
See above. This PR has merge conflicts.
---