RFR: 8281658: Add a security category to the java -XshowSettings option [v8]

Sean Coffey coffeys at openjdk.org
Fri Jul 7 13:47:31 UTC 2023


On Thu, 6 Jul 2023 17:15:20 GMT, Mandy Chung <mchung at openjdk.org> wrote:

> > The changes make me wonder if `-XshowSetting:aardvark` should be an error rather than default to print all settings. Something we should look at again. Same thing for `-XshowSettings:system` on non-Linux, probably should have been a bit more discussion when that was added as its more about the container environment.
> 
> I think throwing an error for an invalid suboption makes sense. `-XshowSetting` without the suboption is default to print all settings, i.e. equivalent to `-XshowSetting:all`. But the switch statment is missing the `all` case. I think we should fix this and error for any invalid option.

fair point Mandy - I hope to change such behavior in a new follow on patch that I'll deliver shortly: JDK-8311653

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

PR Comment: https://git.openjdk.org/jdk/pull/14394#issuecomment-1625434995


More information about the security-dev mailing list