(JDK-8266351) Re: [External] : Re: RFR: 8236671: NullPointerException in JKS keystore [v2]

2021-05-28 Thread Seán Coffey
Thanks for the pointers Will. I've added your details to the JDK-8266351 bug report. https://bugs.openjdk.java.net/browse/JDK-8266351 regards, Sean. On 24/05/2021 18:53, Will Sargent wrote: I have tried to sign up to the bug tracking system (through reset password I think?) but I'm not getting

Re: RFR: 8236671: NullPointerException in JKS keystore [v2]

2021-05-24 Thread Will Sargent
I have tried to sign up to the bug tracking system (through reset password I think?) but I'm not getting an email out, so I can't add to the bug. I have created a test case in Github: https://github.com/wsargent/jca-key-failure/ The stack trace shows the invalid key store entry after saving and

Re: RFR: 8236671: NullPointerException in JKS keystore [v2]

2021-04-30 Thread Seán Coffey
Thanks for the feedback Will. It would be useful if you can provide a testcase and/or add comments to JDK-8266351 on your experience. regards, Sean. On 30/04/2021 17:54, Will Sargent wrote: > KeyStore specification will be tightened up via an

Re: RFR: 8236671: NullPointerException in JKS keystore [v2]

2021-04-30 Thread Will Sargent
> KeyStore specification will be tightened up via another bug record This would be super helpful, as one thing that confuses me is what the relationship is between a key entry and a key alias -- in particular, the existence alias doesn't seem to guarantee a valid entry that can be retrieved. In J

Re: RFR: 8236671: NullPointerException in JKS keystore [v2]

2021-04-30 Thread Sean Coffey
On Wed, 28 Apr 2021 12:39:42 GMT, Sean Coffey wrote: >> Trivial enough change. Improved the exception thrown from JceKeyStore also. > > Sean Coffey has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought i

Re: RFR: 8236671: NullPointerException in JKS keystore [v2]

2021-04-29 Thread Xue-Lei Andrew Fan
On Wed, 28 Apr 2021 12:39:42 GMT, Sean Coffey wrote: >> Trivial enough change. Improved the exception thrown from JceKeyStore also. > > Sean Coffey has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought i

Re: RFR: 8236671: NullPointerException in JKS keystore [v2]

2021-04-28 Thread Sean Coffey
> Trivial enough change. Improved the exception thrown from JceKeyStore also. Sean Coffey has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additiona