On Tue, 7 Mar 2023 10:16:24 GMT, Daniel Fuchs <[email protected]> wrote:
> The HttpClient internal DebugLogger could be simplified if its configuration > was held in a record. Some of the methods in Utils that return a debug logger > could also be simplified/removed. The system property that configures the > debug logging could be extended to choose between System.err, System.out, > System.Logger, or any of these configurations. > Passing -Djdk.internal.httpclient.debug=true still works as before. test/jdk/java/net/httpclient/DebugLoggerTest.java line 242: > 240: out = new RecordingPrintStream(stdOut, true, UTF_8); > 241: System.setErr(err); > 242: System.setOut(out); Should we reset these back to their original streams in a `finally` block? ------------- PR: https://git.openjdk.org/jdk/pull/12900
