Code Review for jdk-updates-jdk11u-dev

Prepared by:shade on Tue Apr 20 09:54:37 CEST 2021
Workspace:/home/shade/trunks/jdk-updates-jdk11u-dev
Compare against: ssh://hg.openjdk.java.net/jdk-updates/jdk11u-dev
Compare against version:a0d9d2e73ef2
Summary of changes: 554 lines changed: 523 ins; 15 del; 16 mod; 3225 unchg
Patch of changes: jdk-updates-jdk11u-dev.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.net.http/share/classes/jdk/internal/net/http/AsyncSSLConnection.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
6 lines changed: 5 ins; 0 del; 1 mod; 112 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.net.http/share/classes/jdk/internal/net/http/AsyncSSLTunnelConnection.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
6 lines changed: 5 ins; 0 del; 1 mod; 129 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.net.http/share/classes/jdk/internal/net/http/Http2Connection.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
53 lines changed: 29 ins; 15 del; 9 mod; 1441 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.net.http/share/classes/jdk/internal/net/http/HttpConnection.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
9 lines changed: 8 ins; 0 del; 1 mod; 488 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.net.http/share/classes/jdk/internal/net/http/PlainHttpConnection.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
5 lines changed: 4 ins; 0 del; 1 mod; 304 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.net.http/share/classes/jdk/internal/net/http/PlainProxyConnection.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
6 lines changed: 5 ins; 0 del; 1 mod; 42 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.net.http/share/classes/jdk/internal/net/http/PlainTunnelingConnection.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
5 lines changed: 4 ins; 0 del; 1 mod; 173 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/net/httpclient/whitebox/java.net.http/jdk/internal/net/http/ConnectionPoolTest.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
2 lines changed: 1 ins; 0 del; 1 mod; 536 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/java/net/httpclient/ProxySelectorTest.java

rev 54110 : 8244205: HTTP/2 tunnel connections through proxy may be reused regardless of which proxy is selected
Summary: The key used in the HTTP/2 connection pool is updated to take into account the proxy address in case of tunnel connections
Reviewed-by: chegar
462 lines changed: 462 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /home/shade/trunks/webrev/webrev.ksh (vers 25.16-hg+openjdk.java.net).