Re: [GitHub] logging-log4j2 issue #176: [LOG4J2-1721] Allow composite configuration for c...

2018-06-24 Thread Phokham Nonava
Hi there, Is there anything I can do to move this issue forward? As I understand Matt Sicker has already reviewed and approved the PR on GitHub. Thanks, Poke On 13.06.18 17:17, Phokham Nonava wrote: Right, for the release-2.x branch I would leave the solution as it is. For the master branch

Re: [GitHub] logging-log4j2 issue #176: [LOG4J2-1721] Allow composite configuration for c...

2018-06-13 Thread Phokham Nonava
Right, for the release-2.x branch I would leave the solution as it is. For the master branch I could create another pull request though. Would it be ok if we could merge this one and I'll create another one for the master branch? On 13.06.18 14:37, Matt Sicker wrote: The master branch is Java

Pull Request #176 Allow composite configuration for context parameter

2018-06-11 Thread Phokham Nonava
Hi all, While working on an enterprise software, I bumped into issue LOG4J2-1721 [1] last week. I took a closer look at the code and have created pull request #176 [2]. Basically, for non-JNDI it allows composite configuration for the "log4jConfiguration" context parameter. I have already mail