vpelikh opened a new pull request, #4154: URL: https://github.com/apache/logging-log4j2/pull/4154
Ports https://github.com/apache/logging-log4j2/pull/3226 to the `main` branch (3.x). ## Changes ### RollingFileManager.java - Rename `releaseRequired` → `outputStreamClosed` / `asyncActionStarted` - Rename `success` → `syncActionSuccess` - Return `outputStreamClosed` instead of `success` / bare `false` — when a rollover's synchronous action fails, the output stream has still been closed, so the log file must be reopened - Semaphore release logic now tracks async action start state explicitly ### RollingFileManagerTest.java - Updated `testSynchronousActionFailure` — removed `initialTime` assertions, verifies file content is preserved after failed rollover - Added `testRolloverOfDeletedFile` — verifies that rolling over a deleted file correctly reopens the log file ### Changelog - Added `src/changelog/.3.x.x/2592_fix_rollover_file_reopen.xml` **Closes:** https://github.com/apache/logging-log4j2/issues/2592 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
