> On 21 Feb 2020, at 17:41, Daniel Fuchs <daniel.fu...@oracle.com> wrote: > > Thanks John! > > I have verified that indeed TLSv1.3 is selected by default, > and updated the fix as per your suggestion: > > http://cr.openjdk.java.net/~dfuchs/webrev_8239052/webrev.01/
Good sleuthing Daniel. Your changes look good, just wondering if they could be factored out into a separate method, say getLong(ByteBuffer), so that the crux of the test logic in onNext would not be slightly obfuscated by this? -Chris.
