Re: [PR] upgrade-log4j2-2.24.2 [solr]

2024-12-16 Thread via GitHub
risdenk commented on PR #2895: URL: https://github.com/apache/solr/pull/2895#issuecomment-2546456414 Thanks @malliaridis will fix in new branch for 2.24.3 and @ppkarwasz for the release w/ the fix -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [PR] upgrade-log4j2-2.24.2 [solr]

2024-12-10 Thread via GitHub
ppkarwasz commented on PR #2895: URL: https://github.com/apache/solr/pull/2895#issuecomment-2530882606 > * [jul-to-slf4j](https://github.com/qos-ch/slf4j/blob/v_2.0.16/jul-to-slf4j/LICENSE.txt) Since you use Log4j Core in your binary distribution, you should consider using the [JUL-t

Re: [PR] upgrade-log4j2-2.24.2 [solr]

2024-12-09 Thread via GitHub
janhoy commented on PR #2895: URL: https://github.com/apache/solr/pull/2895#issuecomment-2530705029 I have no preference when license is ambiguous. Guess whatever is in the download tar for that version is safe. -- This is an automated message from the Apache Git Service. To respond to th

Re: [PR] upgrade-log4j2-2.24.2 [solr]

2024-12-06 Thread via GitHub
ppkarwasz commented on PR #2895: URL: https://github.com/apache/solr/pull/2895#issuecomment-2523609338 @risdenk, I'll cut a `2.24.3` release on Monday. The RC should solve your problems. -- This is an automated message from the Apache Git Service. To respond to the message, please l

Re: [PR] upgrade-log4j2-2.24.2 [solr]

2024-12-06 Thread via GitHub
risdenk commented on PR #2895: URL: https://github.com/apache/solr/pull/2895#issuecomment-2523371681 Looks like we are running into: * https://github.com/apache/logging-log4j2/issues/3234 * https://github.com/apache/logging-log4j2/issues/3283 Once those are fixed (there is a PR i

[PR] upgrade-log4j2-2.24.2 [solr]

2024-12-06 Thread via GitHub
risdenk opened a new pull request, #2895: URL: https://github.com/apache/solr/pull/2895 https://issues.apache.org/jira/browse/SOLR-X This is a follow on to https://github.com/apache/solr/pull/2047 where `biz.aQute.bnd` doesn't need to be upgraded. However running into some