Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
http-client-branch: review comment - several tests should be setting NO_PROXY when testing without proxy http-client-branch
authordfuchs
Fri, 13 Apr 2018 15:06:50 +0100
branchhttp-client-branch
changeset 56426 f39b316f10c9
parent 56425 9236d0fb75b9
child 56427 7f1916397463
http-client-branch: review comment - several tests should be setting NO_PROXY when testing without proxy
test/jdk/java/net/httpclient/InvalidInputStreamSubscriptionRequest.java file | annotate | diff | comparison | revisions
test/jdk/java/net/httpclient/InvalidSubscriptionRequest.java file | annotate | diff | comparison | revisions
test/jdk/java/net/httpclient/ThrowingPublishers.java file | annotate | diff | comparison | revisions
test/jdk/java/net/httpclient/ThrowingPushPromises.java file | annotate | diff | comparison | revisions
test/jdk/java/net/httpclient/ThrowingSubscribers.java file | annotate | diff | comparison | revisions 
--- a/test/jdk/java/net/httpclient/InvalidInputStreamSubscriptionRequest.java	Fri Apr 13 14:59:22 2018 +0100
+++ b/test/jdk/java/net/httpclient/InvalidInputStreamSubscriptionRequest.java	Fri Apr 13 15:06:50 2018 +0100
@@ -202,6 +202,7 @@
 
     HttpClient newHttpClient() {
         return HttpClient.newBuilder()
+                         .proxy(HttpClient.Builder.NO_PROXY)
                          .executor(executor)
                          .sslContext(sslContext)
                          .build();
--- a/test/jdk/java/net/httpclient/InvalidSubscriptionRequest.java	Fri Apr 13 14:59:22 2018 +0100
+++ b/test/jdk/java/net/httpclient/InvalidSubscriptionRequest.java	Fri Apr 13 15:06:50 2018 +0100
@@ -146,6 +146,7 @@
 
     HttpClient newHttpClient() {
         return HttpClient.newBuilder()
+                         .proxy(HttpClient.Builder.NO_PROXY)
                          .executor(executor)
                          .sslContext(sslContext)
                          .build();
--- a/test/jdk/java/net/httpclient/ThrowingPublishers.java	Fri Apr 13 14:59:22 2018 +0100
+++ b/test/jdk/java/net/httpclient/ThrowingPublishers.java	Fri Apr 13 15:06:50 2018 +0100
@@ -223,6 +223,7 @@
     private HttpClient makeNewClient() {
         clientCount.incrementAndGet();
         return TRACKER.track(HttpClient.newBuilder()
+                .proxy(HttpClient.Builder.NO_PROXY)
                 .executor(executor)
                 .sslContext(sslContext)
                 .build());
--- a/test/jdk/java/net/httpclient/ThrowingPushPromises.java	Fri Apr 13 14:59:22 2018 +0100
+++ b/test/jdk/java/net/httpclient/ThrowingPushPromises.java	Fri Apr 13 15:06:50 2018 +0100
@@ -205,6 +205,7 @@
     private HttpClient makeNewClient() {
         clientCount.incrementAndGet();
         return TRACKER.track(HttpClient.newBuilder()
+                .proxy(HttpClient.Builder.NO_PROXY)
                 .executor(executor)
                 .sslContext(sslContext)
                 .build());
--- a/test/jdk/java/net/httpclient/ThrowingSubscribers.java	Fri Apr 13 14:59:22 2018 +0100
+++ b/test/jdk/java/net/httpclient/ThrowingSubscribers.java	Fri Apr 13 15:06:50 2018 +0100
@@ -214,6 +214,7 @@
     private HttpClient makeNewClient() {
         clientCount.incrementAndGet();
         HttpClient client =  HttpClient.newBuilder()
+                .proxy(HttpClient.Builder.NO_PROXY)
                 .executor(executor)
                 .sslContext(sslContext)
                 .build();
jdk-sandbox