On 11/7/2018 1:30 PM, Sean Mullan wrote:
https://bugs.openjdk.java.net/browse/JDK-8213161
http://cr.openjdk.java.net/~xuelei/8212261/webrev.03/
I didn't see a test for SecureCacheResponse - is it possible?
JDK does not have the reference implementation of SecureCacheResponse.
You could also add a test case for IllegalStateExc.
Added in the same test file.
lines 108-113 of HttpsSession.java should be indented 4 more spaces.
Otherwise looks ok to me.
Updated.
New webrev:
http://cr.openjdk.java.net/~xuelei/8212261/webrev.04/
Thanks,
Xuelei