On Fri, 27 Sep 2024 18:48:23 GMT, Jamil Nimeh <[email protected]> wrote:
> This PR corrects a flaw in the StatusResponseManager where it was incorrectly
> swallowing the interrupt status when either an invokeAll was called (spawning
> the threads to fetch each OCSP response) or when attempting to grab the data
> from one of the Futures returned from the fetches.
> Additionally, I made a small change that, in the unlikely event of a
> non-IOException being thrown, only that specific fetch is affected, but other
> successful fetches on different threads can complete and can be added to the
> resulting responseMap.
src/java.base/share/classes/sun/security/ssl/StatusResponseManager.java line
271:
> 269: SSLLogger.fine("Exception during OCSP
> fetch: " +
> 270: cause);
> 271: }
Q: Do we need a 'continue;' here?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21230#discussion_r1785281352
