Request for review: 7198416: Using X500Name in place of CertificateIssuerName and CertificateSubjectName

Jason Uh jason.uh at oracle.com
Tue Oct 30 11:01:10 PDT 2012


Thanks, Sean.

With copyrights updated:
http://cr.openjdk.java.net/~juh/7198416/webrev.02/

Jason

On 10/30/2012 07:56 AM, Sean Mullan wrote:
> Hi Jason,
>
> This looks good. An optional minor comment, but I usually update the copyrights
> to include 2012, as I like to see if the source has been updated and not wait
> for the tool to update them automatically although I know it isn't required.
>
> Anyway, I can push this for you when you are ready.
>
> --Sean
>
> On 10/30/12 4:44 AM, Jason Uh wrote:
>> Updated. Please see instead:
>> http://cr.openjdk.java.net/~juh/7198416/webrev.01/
>>
>> Removed a trailing space and updated certAttributes.html, a sort of
>> cheat sheet for X.509 certificate attributes. Note that the entries for
>> issuerUniqueID and subjectUniqueID have also been updated to reflect the
>> changes that were made in 4647343.
>>
>> Thanks,
>> Jason
>>
>> On 10/29/2012 03:39 PM, Jason Uh wrote:
>>> http://cr.openjdk.java.net/~juh/7198416/webrev.00/
>>>
>>> Because sun.security.x509.CertificateIssuerName and
>>> CertificateSubjectName do not provide any functionality beyond that of
>>> X500Name, they have been removed. X509CertImpl and X509CertInfo have
>>> been modified to use X500Name directly instead of these classes.
>>>
>>> Thanks,
>>> Jason



More information about the security-dev mailing list