oscerd commented on PR #25218: URL: https://github.com/apache/camel/pull/25218#issuecomment-5128022757
Pushed e33199a to green the CI and complete the change: - The tarfile/zipfile `testUnzip/UntarWithEmptyDirectory*` tests rebuilt the archive's directory layout from `CamelFileName`. Since this PR now hardens `CamelFileName` to the stripped base name (Tar/Zip Slip prevention), `getParentFile()` was null for the stripped directory entry → NPE. Updated the tests to rebuild structure from the full entry name that is still exposed on a dedicated header (`CamelTarFileEntryName` for tar, `zipFileName` for zip). - Added a **4.22 upgrade-guide** entry, since setting `CamelFileName` to the base name is a behavior change for routes that relied on the full path being in that header. The rest of the diff (tar/zip/vertx-platform-http) is unchanged. Both dataformat test suites pass locally (tar 16, zip 17). _Claude Code on behalf of Andrea Cosentino (@oscerd)._ -- 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]
