Re: RFR: JDK-8285497: Add system property for Java SE specification maintenance version [v2]
On Thu, 28 Apr 2022 20:15:44 GMT, Joe Darcy wrote: > > Also, don't forget to update the CSR with the new specification text, for > > the record. Otherwise, looks good! > > Will do; I'll update the CSR to reflect that change as well as to be more > explicit about the sequence of values, once that is determined. CSR now updated accordingly. - PR: https://git.openjdk.java.net/jdk/pull/8437
Re: RFR: JDK-8285497: Add system property for Java SE specification maintenance version [v2]
On Thu, 28 Apr 2022 20:07:41 GMT, Joe Darcy wrote: >> src/java.base/share/conf/security/java.policy line 34: >> >>> 32:"java.specification.version", "read"; >>> 33: permission java.util.PropertyPermission >>> 34:"java.specification.maintenance.version", "read"; >> >> For consistency, you should probably also add a hard-coded permission to the >> `sun.security.provider.PolicyFile.initStaticPolicy()` method which is used >> as a fallback if there is a problem parsing the `java.policy` file. I doubt >> this will cause any issues as I don't suspect many/any applications will >> suddenly start reading this property and running with an SM but again it is >> mainly for consistency as all the other default permissions in this file are >> also granted in that method. > > Will do; thanks. Just to double-check, under the current proposal, this > property will be undefined most of the time. I assume it is fine for the > permissions to grant the ability to read a property that is not actually > there. Yes. - PR: https://git.openjdk.java.net/jdk/pull/8437
Re: RFR: JDK-8285497: Add system property for Java SE specification maintenance version [v2]
On Thu, 28 Apr 2022 18:53:11 GMT, Mark Reinhold wrote: > Also, don't forget to update the CSR with the new specification text, for the > record. Otherwise, looks good! Will do; I'll update the CSR to reflect that change as well as to be more explicit about the sequence of values, once that is determined. - PR: https://git.openjdk.java.net/jdk/pull/8437
Re: RFR: JDK-8285497: Add system property for Java SE specification maintenance version [v2]
On Thu, 28 Apr 2022 19:37:00 GMT, Sean Mullan wrote: >> Joe Darcy has updated the pull request with a new target base due to a merge >> or a rebase. The incremental webrev excludes the unrelated changes brought >> in by the merge/rebase. The pull request contains five additional commits >> since the last revision: >> >> - Respond to review feedback. >> - Respond to CSR feedback. >> - Merge branch 'master' into JDK-8285497 >> - Update comment in template. >> - JDK-8285497: Add system property for Java SE specification maintenance >> version > > src/java.base/share/conf/security/java.policy line 34: > >> 32:"java.specification.version", "read"; >> 33: permission java.util.PropertyPermission >> 34:"java.specification.maintenance.version", "read"; > > For consistency, you should probably also add a hard-coded permission to the > `sun.security.provider.PolicyFile.initStaticPolicy()` method which is used as > a fallback if there is a problem parsing the `java.policy` file. I doubt this > will cause any issues as I don't suspect many/any applications will suddenly > start reading this property and running with an SM but again it is mainly for > consistency as all the other default permissions in this file are also > granted in that method. Will do; thanks. Just to double-check, under the current proposal, this property will be undefined most of the time. I assume it is fine for the permissions to grant the ability to read a property that is not actually there. - PR: https://git.openjdk.java.net/jdk/pull/8437