On Tue, 15 Feb 2022 19:51:57 GMT, Weijun Wang <wei...@openjdk.org> wrote:

>> The enhancement adds two extra items in the `getSubjectAlternativeNames()` 
>> output for an OtherName.
>> 
>> It also fix several errors:
>> 1. In `OtherName.java`, `nameValue` should be the value inside `CONTEXT [0]` 
>> without the tag and length bytes.
>> 2. The argument in constructor `extClass.getConstructor(Object.class)` is 
>> suspicious. Maybe it meant `byte[]`.
>
> Weijun Wang has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   retell the formats

New commit pushed. For the openssl style suggestion, I think its major benefit 
is to provide a string format of the type (like `"othername: UPN:"`). In the 
`default` block, it still extracts either the IA5String or the UTF8String. I 
think it's not worth penalizing people putting an IA5String into a UPN. Right?

-------------

PR: https://git.openjdk.java.net/jdk/pull/7167

Reply via email to