Skip to content

Commit f53d051

Browse files
committed
8340312: sun.security.ssl.SSLLogger uses incorrect log level ALL for finest log events
Reviewed-by: wetmore
1 parent 70ebb5e commit f53d051

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/java.base/share/classes/sun/security/ssl/SSLLogger.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ public static void finer(String msg, Object... params) {
179179
}
180180

181181
public static void finest(String msg, Object... params) {
182-
SSLLogger.log(Level.ALL, msg, params);
182+
SSLLogger.log(Level.TRACE, msg, params);
183183
}
184184

185185
private static void log(Level level, String msg, Object... params) {

test/jdk/sun/security/ssl/SSLLogger/DebugPropertyValuesTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323

2424
/**
2525
* @test
26-
* @bug 8350582
26+
* @bug 8350582 8340312
2727
* @library /test/lib /javax/net/ssl/templates
2828
* @summary Correct the parsing of the ssl value in javax.net.debug
2929
* @run junit DebugPropertyValuesTest
@@ -80,6 +80,8 @@ public class DebugPropertyValuesTest extends SSLSocketTemplate {
8080
debugMessages.put("java.security.debug",
8181
List.of("properties\\[.*\\|main\\|.*" + DATE_REGEX + ".*\\]:",
8282
"certpath\\[.*\\|main\\|.*" + DATE_REGEX + ".*\\]:"));
83+
// "ALL" shouldn't be seen as a valid Level
84+
debugMessages.put("javax.net.debug.logger.ALL", List.of("ALL:"));
8385
debugMessages.put("javax.net.debug.logger",
8486
List.of("FINE: adding as trusted certificates",
8587
"FINE: WRITE: TLSv1.3 application_data"));

0 commit comments

Comments
 (0)