RFR: 8331467: ImageReaderFactory can cause a ClassNotFoundException if the default FileSystemProvider is not the system-default provider [v2]

[Alan Bateman]

On Wed, 20 Nov 2024 14:04:54 GMT, jyxzwd <duke at openjdk.org> wrote:

>> Use the built-in file system provider rather than the custom file system provider.
>> Add "public static FileSystemProvider create" method in DefaultFileSystemProvider which is from java8API to be compatible against runtime.
>
> jyxzwd has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - 8331467: Fix JDK-8331467 ImageReaderFactory can cause a ClassNotFoundException if the default FileSystemProvider is not the system-default provider
>  - Revert "8331467: Fix JDK-8331467 ImageReaderFactory can cause a ClassNotFoundException if the default FileSystemProvider is not the system-default provider"
>    
>    This reverts commit 318888339bdc2ee6a632e4850a980d91bff79e79.

I don't think we should be doing this. Did you try the direction of checking the defining class loader that I outlined in the previous comment?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/21997#issuecomment-2488690052