JDK8 Code review request for 7026347: Certificate and X509CRL should have verify(PublicKey key, Provider sigProvider) (original) (raw)
Weijun Wang weijun.wang at oracle.com
Fri Jul 20 06:25:06 UTC 2012
- Previous message (by thread): JDK8 Code review request for 7026347: Certificate and X509CRL should have verify(PublicKey key, Provider sigProvider)
- Next message (by thread): JDK8 Code review request for 7026347: Certificate and X509CRL should have verify(PublicKey key, Provider sigProvider)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi Jason
The existing verify(PublicKey key, String sigProvider) method caches the result and returns it if the same key and sigProvider is used. Is it possible to do the same in your new method?
Also, the new method is very similar to the old one. I have no good idea how to reuse the old code but if you find a way that would be very nice.
As for the CCC, is it really necessary to add these lines?
Note that the specified- Provider object does not have to be registered in the provider list.
This method was added to version 1.8 of the Java Platform
- Standard Edition. In order to maintain backwards compatibility with
- existing service providers, this method cannot be
abstract - and by default throws an
UnsupportedOperationException.
Thanks Max
On 07/20/2012 01:03 PM, Jason Uh wrote:
Hi all,
Please review my fix for 7026347. This change is introduces verify(PublicKey key, Provider sigProvider) methods for Certificate and X509CRL that allow Providers not registered in java.security. Change has been filed with CCC and is accepted. Webrev: http://cr.openjdk.java.net/~juh/7026347/webrev.03/ CR: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=7026347 Thanks, Jason
- Previous message (by thread): JDK8 Code review request for 7026347: Certificate and X509CRL should have verify(PublicKey key, Provider sigProvider)
- Next message (by thread): JDK8 Code review request for 7026347: Certificate and X509CRL should have verify(PublicKey key, Provider sigProvider)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]