GH-550: Upgrade to log4j2 2.23.1

[ihrasko]

Upgrade log4j2 from 2.22.1 to 2.23.1.

[jbonofre]

Did you check class updates in api bundle ? That's actually the most important to check for an update.

[grgrzybek]

True - unfortunately we have to shade some classes. See for example 54f2d59.
When I update Log4j2, I check out related https://github.com/apache/logging-log4j2/ branch and compare (like git diff rel/2.22.1..rel/2.23.1 -- path/to/what/we/shade/in/pax-logging to check if we need to pick up some changes.

I can do this next week.

[grgrzybek]

Yikes, there's a lot of changes in StatusLogger...:

$ git lg rel/2.22.1..rel/2.23.1 -- log4j-api/src/main/java/org/apache/logging/log4j/status/StatusLogger.java
* (2024-03-06 09:34:20 +0100) a4a8e99a7c Fix `StatusLogger` to correctly read `log4j2.StatusLogger.properties` (#2354) <Volkan Yazıcı>
* (2024-03-01 14:49:28 +0100) c5420410df Fix `StatusLogger` log level filtering when debug mode is enabled (#2337, #2338) <Volkan Yazıcı>
* (2024-02-27 21:59:44 +0100) af045db99e Fix `StatusLogger` time-zone issues and stack overflow (#2322) <Volkan Yazıcı>
* (2024-02-12 20:33:42 +0100) 7a7555663c Improve `StatusLogger` javadoc <Volkan Yazıcı>
* (2024-02-12 20:02:48 +0100) e87064f22d Reset the fallback listener on `StatusLogger#reset()` (#2280) <Volkan Yazıcı>
* (2024-02-08 13:29:25 +0100) ce1a3a432e Make `StatusLogger` self-contained and testable (#2249) <Volkan Yazıcı>

[grgrzybek]

No no no no ;) If we just shade without changing, we wouldn't have to do it at all ;)
See for example changes made in StatusLogger after upgrading to 2.11.2: 02bcfca#diff-ff765668251d1511bc1870ffade3dfe4105cb9bc9b1fd8dc57da6e1cfbcc61df

You seem to have simply copied the sources and removed Pax Logging specific changes. Did you run the tests? (maybe they work? :)

[ihrasko]

Thanks for your help. I will try to make it better next time.

[grgrzybek]

No worries - it's not something that can be done without spending a lot of time in Pax Logging ;) Thanks for the PR anyway.
Mind that Pax Logging's first commit was made almost 20 years ago and the amount of technical debt here is enormous. When I first actively joined at 1.10 time to make 1.11 release, I've increased number of integration tests from around 2 to more than 100 - it was a huge undertaking... ;)