On Thu, 29 May 2025 19:06:15 GMT, Sean Coffey <coff...@openjdk.org> wrote:
> Removal of the `+thread` and `+timestamp` options that were used to control > the logging behavior of output from the `java.security.debug` system property. > > > To enhance the security debug logs, the thread and timestamp data should > always be present. This brings it to a par with another important security > debug system property, the TLS debug property: javax.net.debug. Output from > the TLS `javax.net.debug` logs always contains thread and timestamp data. > > This patch remove the `+thread` and `+timestamp` support code and print > thread and timestamp data by default. This enancement is only proposed for > the JDK feature release. Update releases can continue to opt into such data. > > Debug output data from use of the `java.security.debug` property will now > resemble something like the following: > > > > properties[0x10|main|Security.java:122|2025-05-01 14:59:42.859 UTC]: Initial > security property: package.definition=sun.misc.,sun.reflect. > properties[0x10|main|Security.java:122|2025-05-01 14:59:42.859 UTC]: Initial > security property: krb5.kdc.bad.policy=tryLast > > > I've also trimmed back on some of the test case coverage since use of > `+thread` and `+timestamp` options is now redundant with this patch. This pull request has now been integrated. Changeset: 42f48a39 Author: Sean Coffey <coff...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/42f48a39e867ae1683708dda3e158c24a6957180 Stats: 184 lines in 5 files changed: 15 ins; 130 del; 39 mod 8350689: Turn on timestamp and thread metadata by default for java.security.debug Reviewed-by: mullan ------------- PR: https://git.openjdk.org/jdk/pull/25528
