[RFR] [8u] 8144539: Update PKCS11 tests to run with security manager
Andrew Hughes
gnu.andrew at redhat.com
Tue Aug 25 16:46:29 UTC 2020
On 01:40 Tue 25 Aug , Martin Balao wrote:
> Hi,
>
> On Fri, Aug 21, 2020 at 10:43 PM Andrew Hughes <gnu.andrew at redhat.com> wrote:
> >
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8144539
> > Webrev: https://cr.openjdk.java.net/~andrew/openjdk8/8144539/webrev.01/
> >
> >
>
> A few comments:
>
> * There are a few tabs in PKCS11Test.java. Can we replace them with
> white spaces? So we make sure that the code shows properly aligned
> across IDEs.
>
I guess they may have snuck in when I had to bring back that code, as
mentioned in the original e-mail. Should be fixed now with the normalizer
script and jcheck would have caught it on commit.
> * I've seen several changes in a few files such as TestPRF.java,
> TestMasterSecret.java, TestKeyMaterial.java, etc. TestKeyMaterial.java
> was a bit easier for the naked eye. I take from your comment that you
> didn't have any major conflict there beyond the '@modules' line,
> right?
>
I've attached diffs of those three with -b. Most of it is whitespace
changes caused by indenting everything for the try block.
> * test/sun/security/pkcs11/sslecc/ClientJSSEServerJSSE.java and
> test/sun/security/pkcs11/sslecc/JSSEServer.java changes will create a
> conflict with the TLS 1.3 backport; affecting Steps 10, 11 and 15.
>
I've reverted these changes to avoid the conflict as before.
> Otherwise, looks good to me.
>
> With that said, I believe that this backport (which has low-priority)
> should wait until the TLS 1.3 backport (high-priority) is merged.
> Particularly because we are targeting the next release, and should be
> able to merge this week -so it's not that we are blocking SSL-related
> tests for a long time-. Otherwise, we would need to rebase and
> generate + review Steps 10, 11 and 15 again. Note: 15 is still under
> review, but I was expecting to have it ready in the next couple of
> days. That's my view. I'll let other Reviewers decide here.
>
I would prefer we get this in to avoid it being pushed to 8u282, and
it also unblocks the other PKCS#11 fixes. Given it is pretty much ready,
going by your review, I see no reason to delay it further.
New webrev:
https://cr.openjdk.java.net/~andrew/openjdk8/8144539/webrev.02/
It would be easier to see what files conflict with the TLS 1.3 work if
the work was publicly visible. I don't yet see any changes in:
https://hg.openjdk.java.net/jdk8u/jdk8u-jsse-incubator/jdk/
which is a little worrying for something you hope to have merged this
weekend.
> Thanks,
> Martin.-
>
Thanks,
--
Andrew :)
Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)
PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222
More information about the jdk8u-dev
mailing list