On Wed, 8 Apr 2026 04:40:07 GMT, Eirik Bjørsnøs <[email protected]> wrote:
>> Please review this PR which improves validation of unmappable characters in >> strings passed to `ZipFileSystem::getPath` and `ZipPath::resolve`. >> >> Currently, `ZipFileSystem::getPath` and `ZipPath::resolve` throw >> `IllegalArgumentException` when rejecting a path or entry name which cannot >> be encoded with the given charset. This PR adds validation to >> `ZipFileSystem::getBytes` where a failed validation now throws >> `InvalidPathException`. >> >> As far as I can tell, the transitive call sites of `ZipFileSystem::getBytes` >> ends up being `ZipFileSystem::getPath` and `ZipPath::resolve`. >> >> A new test is added in the ZipFS area to verify that the APIs throw >> exceptions according to specifications when faced with unmappable characters >> in primary or secondary path string components. >> >> This PR is similar to a recently integrated change in the `ZipOutputStream` >> API via #30319. >> >> A CSR has been drafted. > > Eirik Bjørsnøs has updated the pull request incrementally with one additional > commit since the last revision: > > Add missing indentation to ZipFileSystem::getBytes try block Thanks for the review Lance, I have finalized the CSR and also drafted a release note for this behavioral change: https://bugs.openjdk.org/browse/JDK-8381845 ------------- PR Comment: https://git.openjdk.org/jdk/pull/30584#issuecomment-4206815267
