> Can I please get a review of this test-only change which addresses the issue > noted in https://bugs.openjdk.org/browse/JDK-8359337? > > On macOS, the JDK by default (without any explicit system properties) picks > up the proxy settings configured on the host. These proxy settings then get > used by the JVM wide `DefaultProxySelector`. That can then result in some > tests, like the ones listed in that issue, to fail. > > The commit in this PR updates these tests to disable the proxy when running > these tests. I've verified that after these changes, these tests now pass on > the host where these tests were failing previously. Additionally I've run > tier2 to make sure nothing else is impacted due to this change.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: Joe's review - add bugid ------------- Changes: - all: https://git.openjdk.org/jdk/pull/26052/files - new: https://git.openjdk.org/jdk/pull/26052/files/8389ac03..17c4febb Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=26052&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26052&range=00-01 Stats: 5 lines in 5 files changed: 0 ins; 0 del; 5 mod Patch: https://git.openjdk.org/jdk/pull/26052.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/26052/head:pull/26052 PR: https://git.openjdk.org/jdk/pull/26052