RFR 8200152: KerberosString should use UTF-8 by default

Xuelei Fan xuelei.fan at oracle.com
Sun Apr 8 02:53:35 UTC 2018


In the current implementation, the map of the property value looks like:
    null    -> false;
    ""      -> false;
    "true"  -> true;
    "false" -> false;
    "not"   -> false

For compatibility, I think you may only want to update the default 
value: null or empty property value, otherwise the behavior does not change.

    null    -> true;
    ""      -> true;
    "true"  -> true;
    "false" -> false;
    "not"   -> false

Per the update, "not" property means true.  It might not be an issue in 
practice.  I will have you make the final decision.  Otherwise, looks 
fine to me.

Xuelei

On 3/22/2018 8:04 PM, Weijun Wang wrote:
> Please take a review of the code change and CSR at
> 
>     CSR: https://bugs.openjdk.java.net/browse/JDK-8200153
>     webrev: http://cr.openjdk.java.net/~weijun/8200152/webrev.00/
> 
> Thanks
> Max
> 


More information about the security-dev mailing list