Code review request: JDK-8046295 - Support Trusted CA Indication extension

Martin Balao mbalao at redhat.com
Thu Jun 15 19:13:55 UTC 2017


Hi Xuelei,

On Thu, Jun 15, 2017 at 3:09 PM, Xuelei Fan <xuelei.fan at oracle.com> wrote:
>
>
> I think more about the new TrustedAuthorityIndicator class.  Maybe, it can
> be replaced with the existing java.util.Map.Entry class (using
> java.util.AbstractMap.SimpleImmutableEntry for the implementation).
>
> ExtendedSSLSession.java
>    List<Map.Entry<Byte, byte[]>> getRequestedTrustedCAs();
>
>
This looks a bit implicit and cryptic to me. I'd go for a more explicit,
extensible and OO interface...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/security-dev/attachments/20170615/453075fc/attachment.htm>


More information about the security-dev mailing list