[9] RFR 8130181: Deprecate java.security.Provider(String, double, String), add Provider(String, String, String)

Wang Weijun weijun.wang at oracle.com
Thu Jun 16 03:33:37 UTC 2016


You used String.valueOf(provider.getVersionStr()) on lines 70 and 861. Why not just provider.getVersionStr()?

For parseVersionStr(), where is the spec for "java.specification.version"? Can we just use regex /(^\d+(\.\d+)?)/ to match the substring and turn it into a double?

--Max

> On Jun 16, 2016, at 8:48 AM, Valerie Peng <valerie.peng at oracle.com> wrote:
> 
> 
> Any one has cycles to review this? Although the number of updated files are not trivial, but the key changes are in only 2 files, i.e.
> src/java.base/share/classes/java/security/Provider.java and src/java.base/share/classes/java/security/AuthProvider.java.
> 
> The rest of files are all very trivial, just changing the double version number to string.
> Instead of hardcoded values, I make most of them refer to the new PROVIDER_VER field of the sun.security.util.SecurityConstants class, so the providers would be automatically updated and use the "java.specification.version" system property value as its version string.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8130181
> Webrev: http://cr.openjdk.java.net/~valeriep/8130181/webrev.00/
> 
> Valerie




More information about the security-dev mailing list