author | chegar |
Mon, 23 Jul 2018 11:47:03 +0100 | |
changeset 51231 | 1d8b1d4eae6a |
parent 50985 | cd41f34e548c |
child 51364 | 31d9e82b2e64 |
child 56842 | 43c3b82728eb |
permissions | -rw-r--r-- |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1 |
/* |
48523
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
2 |
* Copyright (c) 2015, 2018, Oracle and/or its affiliates. All rights reserved. |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
4 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. Oracle designates this |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
8 |
* particular file as subject to the "Classpath" exception as provided |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
9 |
* by Oracle in the LICENSE file that accompanied this code. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
10 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
11 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
12 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
13 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
14 |
* version 2 for more details (a copy is included in the LICENSE file that |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
15 |
* accompanied this code). |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
16 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
17 |
* You should have received a copy of the GNU General Public License version |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
18 |
* 2 along with this work; if not, write to the Free Software Foundation, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
19 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
20 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
21 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
22 |
* or visit www.oracle.com if you need additional information or have any |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
23 |
* questions. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
24 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
25 |
|
49765 | 26 |
package jdk.internal.net.http; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
27 |
|
48083 | 28 |
import java.io.EOFException; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
29 |
import java.io.IOException; |
49765 | 30 |
import java.io.UncheckedIOException; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
31 |
import java.net.InetSocketAddress; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
32 |
import java.net.URI; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
33 |
import java.nio.ByteBuffer; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
34 |
import java.nio.charset.StandardCharsets; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
35 |
import java.util.Iterator; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
36 |
import java.util.LinkedList; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
37 |
import java.util.List; |
49765 | 38 |
import java.util.Locale; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
39 |
import java.util.Map; |
49765 | 40 |
import java.util.Set; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
41 |
import java.util.concurrent.CompletableFuture; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
42 |
import java.util.ArrayList; |
48083 | 43 |
import java.util.Objects; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
44 |
import java.util.concurrent.ConcurrentHashMap; |
48083 | 45 |
import java.util.concurrent.ConcurrentLinkedQueue; |
46 |
import java.util.concurrent.Flow; |
|
47 |
import java.util.function.Function; |
|
48 |
import java.util.function.Supplier; |
|
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
49 |
import javax.net.ssl.SSLEngine; |
49765 | 50 |
import javax.net.ssl.SSLException; |
51 |
import java.net.http.HttpClient; |
|
52 |
import java.net.http.HttpHeaders; |
|
53 |
import jdk.internal.net.http.HttpConnection.HttpPublisher; |
|
54 |
import jdk.internal.net.http.common.FlowTube; |
|
55 |
import jdk.internal.net.http.common.FlowTube.TubeSubscriber; |
|
50681 | 56 |
import jdk.internal.net.http.common.HttpHeadersBuilder; |
49765 | 57 |
import jdk.internal.net.http.common.Log; |
58 |
import jdk.internal.net.http.common.Logger; |
|
59 |
import jdk.internal.net.http.common.MinimalFuture; |
|
60 |
import jdk.internal.net.http.common.SequentialScheduler; |
|
61 |
import jdk.internal.net.http.common.Utils; |
|
62 |
import jdk.internal.net.http.frame.ContinuationFrame; |
|
63 |
import jdk.internal.net.http.frame.DataFrame; |
|
64 |
import jdk.internal.net.http.frame.ErrorFrame; |
|
65 |
import jdk.internal.net.http.frame.FramesDecoder; |
|
66 |
import jdk.internal.net.http.frame.FramesEncoder; |
|
67 |
import jdk.internal.net.http.frame.GoAwayFrame; |
|
68 |
import jdk.internal.net.http.frame.HeaderFrame; |
|
69 |
import jdk.internal.net.http.frame.HeadersFrame; |
|
70 |
import jdk.internal.net.http.frame.Http2Frame; |
|
71 |
import jdk.internal.net.http.frame.MalformedFrame; |
|
72 |
import jdk.internal.net.http.frame.OutgoingHeaders; |
|
73 |
import jdk.internal.net.http.frame.PingFrame; |
|
74 |
import jdk.internal.net.http.frame.PushPromiseFrame; |
|
75 |
import jdk.internal.net.http.frame.ResetFrame; |
|
76 |
import jdk.internal.net.http.frame.SettingsFrame; |
|
77 |
import jdk.internal.net.http.frame.WindowUpdateFrame; |
|
78 |
import jdk.internal.net.http.hpack.Encoder; |
|
79 |
import jdk.internal.net.http.hpack.Decoder; |
|
80 |
import jdk.internal.net.http.hpack.DecodingCallback; |
|
81 |
import static java.nio.charset.StandardCharsets.UTF_8; |
|
82 |
import static jdk.internal.net.http.frame.SettingsFrame.*; |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
83 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
84 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
85 |
* An Http2Connection. Encapsulates the socket(channel) and any SSLEngine used |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
86 |
* over it. Contains an HttpConnection which hides the SocketChannel SSL stuff. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
87 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
88 |
* Http2Connections belong to a Http2ClientImpl, (one of) which belongs |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
89 |
* to a HttpClientImpl. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
90 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
91 |
* Creation cases: |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
92 |
* 1) upgraded HTTP/1.1 plain tcp connection |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
93 |
* 2) prior knowledge directly created plain tcp connection |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
94 |
* 3) directly created HTTP/2 SSL connection which uses ALPN. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
95 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
96 |
* Sending is done by writing directly to underlying HttpConnection object which |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
97 |
* is operating in async mode. No flow control applies on output at this level |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
98 |
* and all writes are just executed as puts to an output Q belonging to HttpConnection |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
99 |
* Flow control is implemented by HTTP/2 protocol itself. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
100 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
101 |
* Hpack header compression |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
102 |
* and outgoing stream creation is also done here, because these operations |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
103 |
* must be synchronized at the socket level. Stream objects send frames simply |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
104 |
* by placing them on the connection's output Queue. sendFrame() is called |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
105 |
* from a higher level (Stream) thread. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
106 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
107 |
* asyncReceive(ByteBuffer) is always called from the selector thread. It assembles |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
108 |
* incoming Http2Frames, and directs them to the appropriate Stream.incoming() |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
109 |
* or handles them directly itself. This thread performs hpack decompression |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
110 |
* and incoming stream creation (Server push). Incoming frames destined for a |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
111 |
* stream are provided by calling Stream.incoming(). |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
112 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
113 |
class Http2Connection { |
48083 | 114 |
|
49765 | 115 |
final Logger debug = Utils.getDebugLogger(this::dbgString, Utils.DEBUG); |
116 |
final static Logger DEBUG_LOGGER = |
|
117 |
Utils.getDebugLogger("Http2Connection"::toString, Utils.DEBUG); |
|
118 |
private final Logger debugHpack = |
|
119 |
Utils.getHpackLogger(this::dbgString, Utils.DEBUG_HPACK); |
|
48083 | 120 |
static final ByteBuffer EMPTY_TRIGGER = ByteBuffer.allocate(0); |
121 |
||
49944 | 122 |
static private final int MAX_CLIENT_STREAM_ID = Integer.MAX_VALUE; // 2147483647 |
123 |
static private final int MAX_SERVER_STREAM_ID = Integer.MAX_VALUE - 1; // 2147483646 |
|
124 |
||
125 |
/** |
|
126 |
* Flag set when no more streams to be opened on this connection. |
|
127 |
* Two cases where it is used. |
|
128 |
* |
|
129 |
* 1. Two connections to the same server were opened concurrently, in which |
|
130 |
* case one of them will be put in the cache, and the second will expire |
|
131 |
* when all its opened streams (which usually should be a single client |
|
132 |
* stream + possibly some additional push-promise server streams) complete. |
|
133 |
* 2. A cached connection reaches its maximum number of streams (~ 2^31-1) |
|
134 |
* either server / or client allocated, in which case it will be taken |
|
135 |
* out of the cache - allowing a new connection to replace it. It will |
|
136 |
* expire when all its still open streams (which could be many) eventually |
|
137 |
* complete. |
|
138 |
*/ |
|
139 |
private boolean finalStream; |
|
48376
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
140 |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
141 |
/* |
49944 | 142 |
* ByteBuffer pooling strategy for HTTP/2 protocol. |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
143 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
144 |
* In general there are 4 points where ByteBuffers are used: |
49944 | 145 |
* - incoming/outgoing frames from/to ByteBuffers plus incoming/outgoing |
146 |
* encrypted data in case of SSL connection. |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
147 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
148 |
* 1. Outgoing frames encoded to ByteBuffers. |
49944 | 149 |
* |
150 |
* Outgoing ByteBuffers are created with required size and frequently |
|
151 |
* small (except DataFrames, etc). At this place no pools at all. All |
|
152 |
* outgoing buffers should eventually be collected by GC. |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
153 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
154 |
* 2. Incoming ByteBuffers (decoded to frames). |
49944 | 155 |
* |
156 |
* Here, total elimination of BB pool is not a good idea. |
|
157 |
* We don't know how many bytes we will receive through network. |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
158 |
* |
49944 | 159 |
* A possible future improvement ( currently not implemented ): |
160 |
* Allocate buffers of reasonable size. The following life of the BB: |
|
161 |
* - If all frames decoded from the BB are other than DataFrame and |
|
162 |
* HeaderFrame (and HeaderFrame subclasses) BB is returned to pool, |
|
163 |
* - If a DataFrame is decoded from the BB. In that case DataFrame refers |
|
164 |
* to sub-buffer obtained by slice(). Such a BB is never returned to the |
|
165 |
* pool and will eventually be GC'ed. |
|
166 |
* - If a HeadersFrame is decoded from the BB. Then header decoding is |
|
167 |
* performed inside processFrame method and the buffer could be release |
|
168 |
* back to pool. |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
169 |
* |
49944 | 170 |
* 3. SSL encrypted buffers ( received ). |
171 |
* |
|
172 |
* The current implementation recycles encrypted buffers read from the |
|
173 |
* channel. The pool of buffers has a maximum size of 3, SocketTube.MAX_BUFFERS, |
|
174 |
* direct buffers which are shared by all connections on a given client. |
|
175 |
* The pool is used by all SSL connections - whether HTTP/1.1 or HTTP/2, |
|
176 |
* but only for SSL encrypted buffers that circulate between the SocketTube |
|
177 |
* Publisher and the SSLFlowDelegate Reader. Limiting the pool to this |
|
178 |
* particular segment allows the use of direct buffers, thus avoiding any |
|
179 |
* additional copy in the NIO socket channel implementation. See |
|
180 |
* HttpClientImpl.SSLDirectBufferSupplier, SocketTube.SSLDirectBufferSource, |
|
181 |
* and SSLTube.recycler. |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
182 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
183 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
184 |
|
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
185 |
// A small class that allows to control frames with respect to the state of |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
186 |
// the connection preface. Any data received before the connection |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
187 |
// preface is sent will be buffered. |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
188 |
private final class FramesController { |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
189 |
volatile boolean prefaceSent; |
48083 | 190 |
volatile List<ByteBuffer> pending; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
191 |
|
48083 | 192 |
boolean processReceivedData(FramesDecoder decoder, ByteBuffer buf) |
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
193 |
throws IOException |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
194 |
{ |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
195 |
// if preface is not sent, buffers data in the pending list |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
196 |
if (!prefaceSent) { |
49765 | 197 |
if (debug.on()) |
198 |
debug.log("Preface not sent: buffering %d", buf.remaining()); |
|
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
199 |
synchronized (this) { |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
200 |
if (!prefaceSent) { |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
201 |
if (pending == null) pending = new ArrayList<>(); |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
202 |
pending.add(buf); |
49765 | 203 |
if (debug.on()) |
204 |
debug.log("there are now %d bytes buffered waiting for preface to be sent" |
|
205 |
+ Utils.remaining(pending) |
|
206 |
); |
|
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
207 |
return false; |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
208 |
} |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
209 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
210 |
} |
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
211 |
|
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
212 |
// Preface is sent. Checks for pending data and flush it. |
48083 | 213 |
// We rely on this method being called from within the Http2TubeSubscriber |
214 |
// scheduler, so we know that no other thread could execute this method |
|
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
215 |
// concurrently while we're here. |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
216 |
// This ensures that later incoming buffers will not |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
217 |
// be processed before we have flushed the pending queue. |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
218 |
// No additional synchronization is therefore necessary here. |
48083 | 219 |
List<ByteBuffer> pending = this.pending; |
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
220 |
this.pending = null; |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
221 |
if (pending != null) { |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
222 |
// flush pending data |
49765 | 223 |
if (debug.on()) debug.log(() -> "Processing buffered data: " |
48083 | 224 |
+ Utils.remaining(pending)); |
225 |
for (ByteBuffer b : pending) { |
|
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
226 |
decoder.decode(b); |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
227 |
} |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
228 |
} |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
229 |
// push the received buffer to the frames decoder. |
48083 | 230 |
if (buf != EMPTY_TRIGGER) { |
49765 | 231 |
if (debug.on()) debug.log("Processing %d", buf.remaining()); |
48083 | 232 |
decoder.decode(buf); |
233 |
} |
|
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
234 |
return true; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
235 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
236 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
237 |
// Mark that the connection preface is sent |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
238 |
void markPrefaceSent() { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
239 |
assert !prefaceSent; |
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
240 |
synchronized (this) { |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
241 |
prefaceSent = true; |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
242 |
} |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
243 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
244 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
245 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
246 |
volatile boolean closed; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
247 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
248 |
//------------------------------------- |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
249 |
final HttpConnection connection; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
250 |
private final Http2ClientImpl client2; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
251 |
private final Map<Integer,Stream<?>> streams = new ConcurrentHashMap<>(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
252 |
private int nextstreamid; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
253 |
private int nextPushStream = 2; |
49944 | 254 |
// actual stream ids are not allocated until the Headers frame is ready |
255 |
// to be sent. The following two fields are updated as soon as a stream |
|
256 |
// is created and assigned to a connection. They are checked before |
|
257 |
// assigning a stream to a connection. |
|
258 |
private int lastReservedClientStreamid = 1; |
|
259 |
private int lastReservedServerStreamid = 0; |
|
50681 | 260 |
private int numReservedClientStreams = 0; // count of current streams |
261 |
private int numReservedServerStreams = 0; // count of current streams |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
262 |
private final Encoder hpackOut; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
263 |
private final Decoder hpackIn; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
264 |
final SettingsFrame clientSettings; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
265 |
private volatile SettingsFrame serverSettings; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
266 |
private final String key; // for HttpClientImpl.connections map |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
267 |
private final FramesDecoder framesDecoder; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
268 |
private final FramesEncoder framesEncoder = new FramesEncoder(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
269 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
270 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
271 |
* Send Window controller for both connection and stream windows. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
272 |
* Each of this connection's Streams MUST use this controller. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
273 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
274 |
private final WindowController windowController = new WindowController(); |
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
275 |
private final FramesController framesController = new FramesController(); |
50681 | 276 |
private final Http2TubeSubscriber subscriber; |
48263
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
277 |
final ConnectionWindowUpdateSender windowUpdater; |
48083 | 278 |
private volatile Throwable cause; |
279 |
private volatile Supplier<ByteBuffer> initial; |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
280 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
281 |
static final int DEFAULT_FRAME_SIZE = 16 * 1024; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
282 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
283 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
284 |
// TODO: need list of control frames from other threads |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
285 |
// that need to be sent |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
286 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
287 |
private Http2Connection(HttpConnection connection, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
288 |
Http2ClientImpl client2, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
289 |
int nextstreamid, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
290 |
String key) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
291 |
this.connection = connection; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
292 |
this.client2 = client2; |
50681 | 293 |
this.subscriber = new Http2TubeSubscriber(client2.client()); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
294 |
this.nextstreamid = nextstreamid; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
295 |
this.key = key; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
296 |
this.clientSettings = this.client2.getClientSettings(); |
48263
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
297 |
this.framesDecoder = new FramesDecoder(this::processFrame, |
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
298 |
clientSettings.getParameter(SettingsFrame.MAX_FRAME_SIZE)); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
299 |
// serverSettings will be updated by server |
51231
1d8b1d4eae6a
8207959: The initial value of SETTINGS_MAX_CONCURRENT_STREAMS should have no limit
chegar
parents:
50985
diff
changeset
|
300 |
this.serverSettings = SettingsFrame.defaultRFCSettings(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
301 |
this.hpackOut = new Encoder(serverSettings.getParameter(HEADER_TABLE_SIZE)); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
302 |
this.hpackIn = new Decoder(clientSettings.getParameter(HEADER_TABLE_SIZE)); |
49765 | 303 |
if (debugHpack.on()) { |
304 |
debugHpack.log("For the record:" + super.toString()); |
|
305 |
debugHpack.log("Decoder created: %s", hpackIn); |
|
306 |
debugHpack.log("Encoder created: %s", hpackOut); |
|
307 |
} |
|
48263
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
308 |
this.windowUpdater = new ConnectionWindowUpdateSender(this, |
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
309 |
client2.getConnectionWindowSize(clientSettings)); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
310 |
} |
46157 | 311 |
|
312 |
/** |
|
313 |
* Case 1) Create from upgraded HTTP/1.1 connection. |
|
50681 | 314 |
* Is ready to use. Can't be SSL. exchange is the Exchange |
46157 | 315 |
* that initiated the connection, whose response will be delivered |
316 |
* on a Stream. |
|
317 |
*/ |
|
48083 | 318 |
private Http2Connection(HttpConnection connection, |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
319 |
Http2ClientImpl client2, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
320 |
Exchange<?> exchange, |
48083 | 321 |
Supplier<ByteBuffer> initial) |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
322 |
throws IOException, InterruptedException |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
323 |
{ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
324 |
this(connection, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
325 |
client2, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
326 |
3, // stream 1 is registered during the upgrade |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
327 |
keyFor(connection)); |
50681 | 328 |
reserveStream(true); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
329 |
Log.logTrace("Connection send window size {0} ", windowController.connectionWindowSize()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
330 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
331 |
Stream<?> initialStream = createStream(exchange); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
332 |
initialStream.registerStream(1); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
333 |
windowController.registerStream(1, getInitialSendWindowSize()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
334 |
initialStream.requestSent(); |
48083 | 335 |
// Upgrading: |
336 |
// set callbacks before sending preface - makes sure anything that |
|
337 |
// might be sent by the server will come our way. |
|
338 |
this.initial = initial; |
|
339 |
connectFlows(connection); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
340 |
sendConnectionPreface(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
341 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
342 |
|
48083 | 343 |
// Used when upgrading an HTTP/1.1 connection to HTTP/2 after receiving |
344 |
// agreement from the server. Async style but completes immediately, because |
|
345 |
// the connection is already connected. |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
346 |
static CompletableFuture<Http2Connection> createAsync(HttpConnection connection, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
347 |
Http2ClientImpl client2, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
348 |
Exchange<?> exchange, |
48083 | 349 |
Supplier<ByteBuffer> initial) |
350 |
{ |
|
43984 | 351 |
return MinimalFuture.supply(() -> new Http2Connection(connection, client2, exchange, initial)); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
352 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
353 |
|
48083 | 354 |
// Requires TLS handshake. So, is really async |
355 |
static CompletableFuture<Http2Connection> createAsync(HttpRequestImpl request, |
|
356 |
Http2ClientImpl h2client) { |
|
357 |
assert request.secure(); |
|
358 |
AbstractAsyncSSLConnection connection = (AbstractAsyncSSLConnection) |
|
359 |
HttpConnection.getConnection(request.getAddress(), |
|
360 |
h2client.client(), |
|
361 |
request, |
|
362 |
HttpClient.Version.HTTP_2); |
|
363 |
||
364 |
return connection.connectAsync() |
|
365 |
.thenCompose(unused -> checkSSLConfig(connection)) |
|
366 |
.thenCompose(notused-> { |
|
367 |
CompletableFuture<Http2Connection> cf = new MinimalFuture<>(); |
|
368 |
try { |
|
369 |
Http2Connection hc = new Http2Connection(request, h2client, connection); |
|
370 |
cf.complete(hc); |
|
371 |
} catch (IOException e) { |
|
372 |
cf.completeExceptionally(e); |
|
373 |
} |
|
374 |
return cf; } ); |
|
375 |
} |
|
376 |
||
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
377 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
378 |
* Cases 2) 3) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
379 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
380 |
* request is request to be sent. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
381 |
*/ |
48083 | 382 |
private Http2Connection(HttpRequestImpl request, |
383 |
Http2ClientImpl h2client, |
|
384 |
HttpConnection connection) |
|
385 |
throws IOException |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
386 |
{ |
48083 | 387 |
this(connection, |
388 |
h2client, |
|
389 |
1, |
|
390 |
keyFor(request.uri(), request.proxy())); |
|
391 |
||
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
392 |
Log.logTrace("Connection send window size {0} ", windowController.connectionWindowSize()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
393 |
|
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
394 |
// safe to resume async reading now. |
48083 | 395 |
connectFlows(connection); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
396 |
sendConnectionPreface(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
397 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
398 |
|
48083 | 399 |
private void connectFlows(HttpConnection connection) { |
400 |
FlowTube tube = connection.getConnectionFlow(); |
|
401 |
// Connect the flow to our Http2TubeSubscriber: |
|
402 |
tube.connectFlows(connection.publisher(), subscriber); |
|
403 |
} |
|
404 |
||
405 |
final HttpClientImpl client() { |
|
406 |
return client2.client(); |
|
407 |
} |
|
408 |
||
49944 | 409 |
// call these before assigning a request/stream to a connection |
410 |
// if false returned then a new Http2Connection is required |
|
411 |
// if true, the the stream may be assigned to this connection |
|
50681 | 412 |
// for server push, if false returned, then the stream should be cancelled |
413 |
synchronized boolean reserveStream(boolean clientInitiated) throws IOException { |
|
49944 | 414 |
if (finalStream) { |
415 |
return false; |
|
416 |
} |
|
417 |
if (clientInitiated && (lastReservedClientStreamid + 2) >= MAX_CLIENT_STREAM_ID) { |
|
418 |
setFinalStream(); |
|
419 |
client2.deleteConnection(this); |
|
420 |
return false; |
|
421 |
} else if (!clientInitiated && (lastReservedServerStreamid + 2) >= MAX_SERVER_STREAM_ID) { |
|
422 |
setFinalStream(); |
|
423 |
client2.deleteConnection(this); |
|
424 |
return false; |
|
425 |
} |
|
426 |
if (clientInitiated) |
|
427 |
lastReservedClientStreamid+=2; |
|
428 |
else |
|
429 |
lastReservedServerStreamid+=2; |
|
50681 | 430 |
|
431 |
assert numReservedClientStreams >= 0; |
|
432 |
assert numReservedServerStreams >= 0; |
|
51231
1d8b1d4eae6a
8207959: The initial value of SETTINGS_MAX_CONCURRENT_STREAMS should have no limit
chegar
parents:
50985
diff
changeset
|
433 |
if (clientInitiated &&numReservedClientStreams >= maxConcurrentClientInitiatedStreams()) { |
50681 | 434 |
throw new IOException("too many concurrent streams"); |
435 |
} else if (clientInitiated) { |
|
436 |
numReservedClientStreams++; |
|
437 |
} |
|
51231
1d8b1d4eae6a
8207959: The initial value of SETTINGS_MAX_CONCURRENT_STREAMS should have no limit
chegar
parents:
50985
diff
changeset
|
438 |
if (!clientInitiated && numReservedServerStreams >= maxConcurrentServerInitiatedStreams()) { |
50681 | 439 |
return false; |
440 |
} else if (!clientInitiated) { |
|
441 |
numReservedServerStreams++; |
|
442 |
} |
|
49944 | 443 |
return true; |
444 |
} |
|
445 |
||
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
446 |
/** |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
447 |
* Throws an IOException if h2 was not negotiated |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
448 |
*/ |
48083 | 449 |
private static CompletableFuture<?> checkSSLConfig(AbstractAsyncSSLConnection aconn) { |
450 |
assert aconn.isSecure(); |
|
451 |
||
452 |
Function<String, CompletableFuture<Void>> checkAlpnCF = (alpn) -> { |
|
453 |
CompletableFuture<Void> cf = new MinimalFuture<>(); |
|
454 |
SSLEngine engine = aconn.getEngine(); |
|
455 |
assert Objects.equals(alpn, engine.getApplicationProtocol()); |
|
456 |
||
49765 | 457 |
DEBUG_LOGGER.log("checkSSLConfig: alpn: %s", alpn ); |
48083 | 458 |
|
459 |
if (alpn == null || !alpn.equals("h2")) { |
|
460 |
String msg; |
|
461 |
if (alpn == null) { |
|
462 |
Log.logSSL("ALPN not supported"); |
|
463 |
msg = "ALPN not supported"; |
|
464 |
} else { |
|
465 |
switch (alpn) { |
|
466 |
case "": |
|
467 |
Log.logSSL(msg = "No ALPN negotiated"); |
|
468 |
break; |
|
469 |
case "http/1.1": |
|
470 |
Log.logSSL( msg = "HTTP/1.1 ALPN returned"); |
|
471 |
break; |
|
472 |
default: |
|
473 |
Log.logSSL(msg = "Unexpected ALPN: " + alpn); |
|
474 |
cf.completeExceptionally(new IOException(msg)); |
|
475 |
} |
|
476 |
} |
|
477 |
cf.completeExceptionally(new ALPNException(msg, aconn)); |
|
478 |
return cf; |
|
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
479 |
} |
48083 | 480 |
cf.complete(null); |
481 |
return cf; |
|
482 |
}; |
|
483 |
||
49765 | 484 |
return aconn.getALPN() |
485 |
.whenComplete((r,t) -> { |
|
486 |
if (t != null && t instanceof SSLException) { |
|
487 |
// something went wrong during the initial handshake |
|
488 |
// close the connection |
|
489 |
aconn.close(); |
|
490 |
} |
|
491 |
}) |
|
492 |
.thenCompose(checkAlpnCF); |
|
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
493 |
} |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
494 |
|
49944 | 495 |
synchronized boolean finalStream() { |
496 |
return finalStream; |
|
48376
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
497 |
} |
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
498 |
|
49944 | 499 |
/** |
500 |
* Mark this connection so no more streams created on it and it will close when |
|
501 |
* all are complete. |
|
502 |
*/ |
|
503 |
synchronized void setFinalStream() { |
|
504 |
finalStream = true; |
|
48376
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
505 |
} |
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
506 |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
507 |
static String keyFor(HttpConnection connection) { |
49765 | 508 |
boolean isProxy = connection.isProxied(); // tunnel or plain clear connection through proxy |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
509 |
boolean isSecure = connection.isSecure(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
510 |
InetSocketAddress addr = connection.address(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
511 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
512 |
return keyString(isSecure, isProxy, addr.getHostString(), addr.getPort()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
513 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
514 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
515 |
static String keyFor(URI uri, InetSocketAddress proxy) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
516 |
boolean isSecure = uri.getScheme().equalsIgnoreCase("https"); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
517 |
boolean isProxy = proxy != null; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
518 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
519 |
String host; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
520 |
int port; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
521 |
|
49765 | 522 |
if (proxy != null && !isSecure) { |
523 |
// clear connection through proxy: use |
|
524 |
// proxy host / proxy port |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
525 |
host = proxy.getHostString(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
526 |
port = proxy.getPort(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
527 |
} else { |
49765 | 528 |
// either secure tunnel connection through proxy |
529 |
// or direct connection to host, but in either |
|
530 |
// case only that host can be reached through |
|
531 |
// the connection: use target host / target port |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
532 |
host = uri.getHost(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
533 |
port = uri.getPort(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
534 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
535 |
return keyString(isSecure, isProxy, host, port); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
536 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
537 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
538 |
// {C,S}:{H:P}:host:port |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
539 |
// C indicates clear text connection "http" |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
540 |
// S indicates secure "https" |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
541 |
// H indicates host (direct) connection |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
542 |
// P indicates proxy |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
543 |
// Eg: "S:H:foo.com:80" |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
544 |
static String keyString(boolean secure, boolean proxy, String host, int port) { |
48376
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
545 |
if (secure && port == -1) |
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
546 |
port = 443; |
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
547 |
else if (!secure && port == -1) |
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
548 |
port = 80; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
549 |
return (secure ? "S:" : "C:") + (proxy ? "P:" : "H:") + host + ":" + port; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
550 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
551 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
552 |
String key() { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
553 |
return this.key; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
554 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
555 |
|
48376
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
556 |
boolean offerConnection() { |
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
557 |
return client2.offerConnection(this); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
558 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
559 |
|
48083 | 560 |
private HttpPublisher publisher() { |
561 |
return connection.publisher(); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
562 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
563 |
|
48083 | 564 |
private void decodeHeaders(HeaderFrame frame, DecodingCallback decoder) |
565 |
throws IOException |
|
566 |
{ |
|
49765 | 567 |
if (debugHpack.on()) debugHpack.log("decodeHeaders(%s)", decoder); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
568 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
569 |
boolean endOfHeaders = frame.getFlag(HeaderFrame.END_HEADERS); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
570 |
|
48083 | 571 |
List<ByteBuffer> buffers = frame.getHeaderBlock(); |
572 |
int len = buffers.size(); |
|
573 |
for (int i = 0; i < len; i++) { |
|
574 |
ByteBuffer b = buffers.get(i); |
|
575 |
hpackIn.decode(b, endOfHeaders && (i == len - 1), decoder); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
576 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
577 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
578 |
|
48083 | 579 |
final int getInitialSendWindowSize() { |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
580 |
return serverSettings.getParameter(INITIAL_WINDOW_SIZE); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
581 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
582 |
|
51231
1d8b1d4eae6a
8207959: The initial value of SETTINGS_MAX_CONCURRENT_STREAMS should have no limit
chegar
parents:
50985
diff
changeset
|
583 |
final int maxConcurrentClientInitiatedStreams() { |
50681 | 584 |
return serverSettings.getParameter(MAX_CONCURRENT_STREAMS); |
585 |
} |
|
586 |
||
51231
1d8b1d4eae6a
8207959: The initial value of SETTINGS_MAX_CONCURRENT_STREAMS should have no limit
chegar
parents:
50985
diff
changeset
|
587 |
final int maxConcurrentServerInitiatedStreams() { |
50681 | 588 |
return clientSettings.getParameter(MAX_CONCURRENT_STREAMS); |
589 |
} |
|
590 |
||
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
591 |
void close() { |
48376
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
592 |
Log.logTrace("Closing HTTP/2 connection: to {0}", connection.address()); |
49765 | 593 |
GoAwayFrame f = new GoAwayFrame(0, |
594 |
ErrorFrame.NO_ERROR, |
|
595 |
"Requested by user".getBytes(UTF_8)); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
596 |
// TODO: set last stream. For now zero ok. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
597 |
sendFrame(f); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
598 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
599 |
|
48083 | 600 |
long count; |
601 |
final void asyncReceive(ByteBuffer buffer) { |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
602 |
// We don't need to read anything and |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
603 |
// we don't want to send anything back to the server |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
604 |
// until the connection preface has been sent. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
605 |
// Therefore we're going to wait if needed before reading |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
606 |
// (and thus replying) to anything. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
607 |
// Starting to reply to something (e.g send an ACK to a |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
608 |
// SettingsFrame sent by the server) before the connection |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
609 |
// preface is fully sent might result in the server |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
610 |
// sending a GOAWAY frame with 'invalid_preface'. |
48083 | 611 |
// |
612 |
// Note: asyncReceive is only called from the Http2TubeSubscriber |
|
613 |
// sequential scheduler. |
|
614 |
try { |
|
615 |
Supplier<ByteBuffer> bs = initial; |
|
616 |
// ensure that we always handle the initial buffer first, |
|
617 |
// if any. |
|
618 |
if (bs != null) { |
|
619 |
initial = null; |
|
620 |
ByteBuffer b = bs.get(); |
|
621 |
if (b.hasRemaining()) { |
|
622 |
long c = ++count; |
|
49765 | 623 |
if (debug.on()) |
624 |
debug.log(() -> "H2 Receiving Initial(" + c +"): " + b.remaining()); |
|
48083 | 625 |
framesController.processReceivedData(framesDecoder, b); |
626 |
} |
|
627 |
} |
|
628 |
ByteBuffer b = buffer; |
|
629 |
// the Http2TubeSubscriber scheduler ensures that the order of incoming |
|
630 |
// buffers is preserved. |
|
631 |
if (b == EMPTY_TRIGGER) { |
|
49765 | 632 |
if (debug.on()) debug.log("H2 Received EMPTY_TRIGGER"); |
48083 | 633 |
boolean prefaceSent = framesController.prefaceSent; |
634 |
assert prefaceSent; |
|
635 |
// call framesController.processReceivedData to potentially |
|
636 |
// trigger the processing of all the data buffered there. |
|
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
637 |
framesController.processReceivedData(framesDecoder, buffer); |
49765 | 638 |
if (debug.on()) debug.log("H2 processed buffered data"); |
48083 | 639 |
} else { |
640 |
long c = ++count; |
|
49765 | 641 |
if (debug.on()) |
642 |
debug.log("H2 Receiving(%d): %d", c, b.remaining()); |
|
48083 | 643 |
framesController.processReceivedData(framesDecoder, buffer); |
49765 | 644 |
if (debug.on()) debug.log("H2 processed(%d)", c); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
645 |
} |
48083 | 646 |
} catch (Throwable e) { |
647 |
String msg = Utils.stackTrace(e); |
|
648 |
Log.logTrace(msg); |
|
649 |
shutdown(e); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
650 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
651 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
652 |
|
48083 | 653 |
Throwable getRecordedCause() { |
654 |
return cause; |
|
655 |
} |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
656 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
657 |
void shutdown(Throwable t) { |
49765 | 658 |
if (debug.on()) debug.log(() -> "Shutting down h2c (closed="+closed+"): " + t); |
48083 | 659 |
if (closed == true) return; |
660 |
synchronized (this) { |
|
661 |
if (closed == true) return; |
|
662 |
closed = true; |
|
663 |
} |
|
50681 | 664 |
if (Log.errors()) { |
665 |
if (!(t instanceof EOFException) || isActive()) { |
|
666 |
Log.logError(t); |
|
667 |
} else if (t != null) { |
|
668 |
Log.logError("Shutting down connection: {0}", t.getMessage()); |
|
669 |
} |
|
670 |
} |
|
48083 | 671 |
Throwable initialCause = this.cause; |
672 |
if (initialCause == null) this.cause = t; |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
673 |
client2.deleteConnection(this); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
674 |
List<Stream<?>> c = new LinkedList<>(streams.values()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
675 |
for (Stream<?> s : c) { |
50985
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
676 |
try { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
677 |
s.connectionClosing(t); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
678 |
} catch (Throwable e) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
679 |
Log.logError("Failed to close stream {0}: {1}", s.streamid, e); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
680 |
} |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
681 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
682 |
connection.close(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
683 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
684 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
685 |
/** |
48523
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
686 |
* Streams initiated by a client MUST use odd-numbered stream |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
687 |
* identifiers; those initiated by the server MUST use even-numbered |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
688 |
* stream identifiers. |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
689 |
*/ |
49765 | 690 |
private static final boolean isServerInitiatedStream(int streamid) { |
48523
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
691 |
return (streamid & 0x1) == 0; |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
692 |
} |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
693 |
|
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
694 |
/** |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
695 |
* Handles stream 0 (common) frames that apply to whole connection and passes |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
696 |
* other stream specific frames to that Stream object. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
697 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
698 |
* Invokes Stream.incoming() which is expected to process frame without |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
699 |
* blocking. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
700 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
701 |
void processFrame(Http2Frame frame) throws IOException { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
702 |
Log.logFrames(frame, "IN"); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
703 |
int streamid = frame.streamid(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
704 |
if (frame instanceof MalformedFrame) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
705 |
Log.logError(((MalformedFrame) frame).getMessage()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
706 |
if (streamid == 0) { |
48083 | 707 |
framesDecoder.close("Malformed frame on stream 0"); |
708 |
protocolError(((MalformedFrame) frame).getErrorCode(), |
|
709 |
((MalformedFrame) frame).getMessage()); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
710 |
} else { |
49765 | 711 |
if (debug.on()) |
712 |
debug.log(() -> "Reset stream: " + ((MalformedFrame) frame).getMessage()); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
713 |
resetStream(streamid, ((MalformedFrame) frame).getErrorCode()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
714 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
715 |
return; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
716 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
717 |
if (streamid == 0) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
718 |
handleConnectionFrame(frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
719 |
} else { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
720 |
if (frame instanceof SettingsFrame) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
721 |
// The stream identifier for a SETTINGS frame MUST be zero |
48083 | 722 |
framesDecoder.close( |
723 |
"The stream identifier for a SETTINGS frame MUST be zero"); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
724 |
protocolError(GoAwayFrame.PROTOCOL_ERROR); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
725 |
return; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
726 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
727 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
728 |
Stream<?> stream = getStream(streamid); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
729 |
if (stream == null) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
730 |
// Should never receive a frame with unknown stream id |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
731 |
|
48083 | 732 |
if (frame instanceof HeaderFrame) { |
733 |
// always decode the headers as they may affect |
|
734 |
// connection-level HPACK decoding state |
|
49765 | 735 |
DecodingCallback decoder = new ValidatingHeadersConsumer(); |
736 |
try { |
|
737 |
decodeHeaders((HeaderFrame) frame, decoder); |
|
738 |
} catch (UncheckedIOException e) { |
|
739 |
protocolError(ResetFrame.PROTOCOL_ERROR, e.getMessage()); |
|
740 |
return; |
|
741 |
} |
|
48083 | 742 |
} |
743 |
||
48523
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
744 |
if (!(frame instanceof ResetFrame)) { |
50985
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
745 |
if (frame instanceof DataFrame) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
746 |
dropDataFrame((DataFrame)frame); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
747 |
} |
49765 | 748 |
if (isServerInitiatedStream(streamid)) { |
48523
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
749 |
if (streamid < nextPushStream) { |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
750 |
// trailing data on a cancelled push promise stream, |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
751 |
// reset will already have been sent, ignore |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
752 |
Log.logTrace("Ignoring cancelled push promise frame " + frame); |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
753 |
} else { |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
754 |
resetStream(streamid, ResetFrame.PROTOCOL_ERROR); |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
755 |
} |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
756 |
} else if (streamid >= nextstreamid) { |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
757 |
// otherwise the stream has already been reset/closed |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
758 |
resetStream(streamid, ResetFrame.PROTOCOL_ERROR); |
b95b08f3e1a8
8194883: Unhandleable Push Promises should be cancelled
chegar
parents:
48376
diff
changeset
|
759 |
} |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
760 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
761 |
return; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
762 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
763 |
if (frame instanceof PushPromiseFrame) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
764 |
PushPromiseFrame pp = (PushPromiseFrame)frame; |
49765 | 765 |
try { |
766 |
handlePushPromise(stream, pp); |
|
767 |
} catch (UncheckedIOException e) { |
|
768 |
protocolError(ResetFrame.PROTOCOL_ERROR, e.getMessage()); |
|
769 |
return; |
|
770 |
} |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
771 |
} else if (frame instanceof HeaderFrame) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
772 |
// decode headers (or continuation) |
49765 | 773 |
try { |
774 |
decodeHeaders((HeaderFrame) frame, stream.rspHeadersConsumer()); |
|
775 |
} catch (UncheckedIOException e) { |
|
776 |
protocolError(ResetFrame.PROTOCOL_ERROR, e.getMessage()); |
|
777 |
return; |
|
778 |
} |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
779 |
stream.incoming(frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
780 |
} else { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
781 |
stream.incoming(frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
782 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
783 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
784 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
785 |
|
50985
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
786 |
final void dropDataFrame(DataFrame df) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
787 |
if (closed) return; |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
788 |
if (debug.on()) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
789 |
debug.log("Dropping data frame for stream %d (%d payload bytes)", |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
790 |
df.streamid(), df.payloadLength()); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
791 |
} |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
792 |
ensureWindowUpdated(df); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
793 |
} |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
794 |
|
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
795 |
final void ensureWindowUpdated(DataFrame df) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
796 |
try { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
797 |
if (closed) return; |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
798 |
int length = df.payloadLength(); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
799 |
if (length > 0) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
800 |
windowUpdater.update(length); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
801 |
} |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
802 |
} catch(Throwable t) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
803 |
Log.logError("Unexpected exception while updating window: {0}", (Object)t); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
804 |
} |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
805 |
} |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
806 |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
807 |
private <T> void handlePushPromise(Stream<T> parent, PushPromiseFrame pp) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
808 |
throws IOException |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
809 |
{ |
48083 | 810 |
// always decode the headers as they may affect connection-level HPACK |
811 |
// decoding state |
|
49765 | 812 |
HeaderDecoder decoder = new HeaderDecoder(); |
48083 | 813 |
decodeHeaders(pp, decoder); |
814 |
||
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
815 |
HttpRequestImpl parentReq = parent.request; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
816 |
int promisedStreamid = pp.getPromisedStream(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
817 |
if (promisedStreamid != nextPushStream) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
818 |
resetStream(promisedStreamid, ResetFrame.PROTOCOL_ERROR); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
819 |
return; |
49944 | 820 |
} else if (!reserveStream(false)) { |
821 |
resetStream(promisedStreamid, ResetFrame.REFUSED_STREAM); |
|
822 |
return; |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
823 |
} else { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
824 |
nextPushStream += 2; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
825 |
} |
48083 | 826 |
|
50681 | 827 |
HttpHeaders headers = decoder.headers(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
828 |
HttpRequestImpl pushReq = HttpRequestImpl.createPushRequest(parentReq, headers); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
829 |
Exchange<T> pushExch = new Exchange<>(pushReq, parent.exchange.multi); |
49765 | 830 |
Stream.PushedStream<T> pushStream = createPushStream(parent, pushExch); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
831 |
pushExch.exchImpl = pushStream; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
832 |
pushStream.registerStream(promisedStreamid); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
833 |
parent.incoming_pushPromise(pushReq, pushStream); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
834 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
835 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
836 |
private void handleConnectionFrame(Http2Frame frame) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
837 |
throws IOException |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
838 |
{ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
839 |
switch (frame.type()) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
840 |
case SettingsFrame.TYPE: |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
841 |
handleSettings((SettingsFrame)frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
842 |
break; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
843 |
case PingFrame.TYPE: |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
844 |
handlePing((PingFrame)frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
845 |
break; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
846 |
case GoAwayFrame.TYPE: |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
847 |
handleGoAway((GoAwayFrame)frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
848 |
break; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
849 |
case WindowUpdateFrame.TYPE: |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
850 |
handleWindowUpdate((WindowUpdateFrame)frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
851 |
break; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
852 |
default: |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
853 |
protocolError(ErrorFrame.PROTOCOL_ERROR); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
854 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
855 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
856 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
857 |
void resetStream(int streamid, int code) throws IOException { |
50681 | 858 |
try { |
859 |
if (connection.channel().isOpen()) { |
|
860 |
// no need to try & send a reset frame if the |
|
861 |
// connection channel is already closed. |
|
862 |
Log.logError( |
|
863 |
"Resetting stream {0,number,integer} with error code {1,number,integer}", |
|
864 |
streamid, code); |
|
865 |
ResetFrame frame = new ResetFrame(streamid, code); |
|
866 |
sendFrame(frame); |
|
867 |
} else if (debug.on()) { |
|
868 |
debug.log("Channel already closed, no need to reset stream %d", |
|
869 |
streamid); |
|
870 |
} |
|
871 |
} finally { |
|
872 |
decrementStreamsCount(streamid); |
|
873 |
closeStream(streamid); |
|
874 |
} |
|
875 |
} |
|
876 |
||
877 |
// reduce count of streams by 1 if stream still exists |
|
878 |
synchronized void decrementStreamsCount(int streamid) { |
|
879 |
Stream<?> s = streams.get(streamid); |
|
880 |
if (s == null || !s.deRegister()) |
|
881 |
return; |
|
882 |
if (streamid % 2 == 1) { |
|
883 |
numReservedClientStreams--; |
|
884 |
assert numReservedClientStreams >= 0 : |
|
885 |
"negative client stream count for stream=" + streamid; |
|
886 |
} else { |
|
887 |
numReservedServerStreams--; |
|
888 |
assert numReservedServerStreams >= 0 : |
|
889 |
"negative server stream count for stream=" + streamid; |
|
890 |
} |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
891 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
892 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
893 |
void closeStream(int streamid) { |
49765 | 894 |
if (debug.on()) debug.log("Closed stream %d", streamid); |
50681 | 895 |
boolean isClient = (streamid % 2) == 1; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
896 |
Stream<?> s = streams.remove(streamid); |
48083 | 897 |
if (s != null) { |
898 |
// decrement the reference count on the HttpClientImpl |
|
899 |
// to allow the SelectorManager thread to exit if no |
|
900 |
// other operation is pending and the facade is no |
|
901 |
// longer referenced. |
|
49765 | 902 |
client().streamUnreference(); |
48083 | 903 |
} |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
904 |
// ## Remove s != null. It is a hack for delayed cancellation,reset |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
905 |
if (s != null && !(s instanceof Stream.PushedStream)) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
906 |
// Since PushStreams have no request body, then they have no |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
907 |
// corresponding entry in the window controller. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
908 |
windowController.removeStream(streamid); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
909 |
} |
49944 | 910 |
if (finalStream() && streams.isEmpty()) { |
48376
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
911 |
// should be only 1 stream, but there might be more if server push |
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
912 |
close(); |
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
913 |
} |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
914 |
} |
48376
41ae5c69b09c
8192966: HttpClient should reuse TCP connection for h2c connections
michaelm
parents:
48263
diff
changeset
|
915 |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
916 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
917 |
* Increments this connection's send Window by the amount in the given frame. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
918 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
919 |
private void handleWindowUpdate(WindowUpdateFrame f) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
920 |
throws IOException |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
921 |
{ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
922 |
int amount = f.getUpdate(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
923 |
if (amount <= 0) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
924 |
// ## temporarily disable to workaround a bug in Jetty where it |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
925 |
// ## sends Window updates with a 0 update value. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
926 |
//protocolError(ErrorFrame.PROTOCOL_ERROR); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
927 |
} else { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
928 |
boolean success = windowController.increaseConnectionWindow(amount); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
929 |
if (!success) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
930 |
protocolError(ErrorFrame.FLOW_CONTROL_ERROR); // overflow |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
931 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
932 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
933 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
934 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
935 |
private void protocolError(int errorCode) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
936 |
throws IOException |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
937 |
{ |
48083 | 938 |
protocolError(errorCode, null); |
939 |
} |
|
940 |
||
941 |
private void protocolError(int errorCode, String msg) |
|
942 |
throws IOException |
|
943 |
{ |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
944 |
GoAwayFrame frame = new GoAwayFrame(0, errorCode); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
945 |
sendFrame(frame); |
48083 | 946 |
shutdown(new IOException("protocol error" + (msg == null?"":(": " + msg)))); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
947 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
948 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
949 |
private void handleSettings(SettingsFrame frame) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
950 |
throws IOException |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
951 |
{ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
952 |
assert frame.streamid() == 0; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
953 |
if (!frame.getFlag(SettingsFrame.ACK)) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
954 |
int newWindowSize = frame.getParameter(INITIAL_WINDOW_SIZE); |
49765 | 955 |
if (newWindowSize != -1) { |
956 |
int oldWindowSize = serverSettings.getParameter(INITIAL_WINDOW_SIZE); |
|
957 |
int diff = newWindowSize - oldWindowSize; |
|
958 |
if (diff != 0) { |
|
959 |
windowController.adjustActiveStreams(diff); |
|
960 |
} |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
961 |
} |
49765 | 962 |
|
963 |
serverSettings.update(frame); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
964 |
sendFrame(new SettingsFrame(SettingsFrame.ACK)); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
965 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
966 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
967 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
968 |
private void handlePing(PingFrame frame) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
969 |
throws IOException |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
970 |
{ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
971 |
frame.setFlag(PingFrame.ACK); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
972 |
sendUnorderedFrame(frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
973 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
974 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
975 |
private void handleGoAway(GoAwayFrame frame) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
976 |
throws IOException |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
977 |
{ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
978 |
shutdown(new IOException( |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
979 |
String.valueOf(connection.channel().getLocalAddress()) |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
980 |
+": GOAWAY received")); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
981 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
982 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
983 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
984 |
* Max frame size we are allowed to send |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
985 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
986 |
public int getMaxSendFrameSize() { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
987 |
int param = serverSettings.getParameter(MAX_FRAME_SIZE); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
988 |
if (param == -1) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
989 |
param = DEFAULT_FRAME_SIZE; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
990 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
991 |
return param; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
992 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
993 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
994 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
995 |
* Max frame size we will receive |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
996 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
997 |
public int getMaxReceiveFrameSize() { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
998 |
return clientSettings.getParameter(MAX_FRAME_SIZE); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
999 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1000 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1001 |
private static final String CLIENT_PREFACE = "PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n"; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1002 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1003 |
private static final byte[] PREFACE_BYTES = |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1004 |
CLIENT_PREFACE.getBytes(StandardCharsets.ISO_8859_1); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1005 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1006 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1007 |
* Sends Connection preface and Settings frame with current preferred |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1008 |
* values |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1009 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1010 |
private void sendConnectionPreface() throws IOException { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1011 |
Log.logTrace("{0}: start sending connection preface to {1}", |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1012 |
connection.channel().getLocalAddress(), |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1013 |
connection.address()); |
48263
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
1014 |
SettingsFrame sf = new SettingsFrame(clientSettings); |
48083 | 1015 |
ByteBuffer buf = framesEncoder.encodeConnectionPreface(PREFACE_BYTES, sf); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1016 |
Log.logFrames(sf, "OUT"); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1017 |
// send preface bytes and SettingsFrame together |
48083 | 1018 |
HttpPublisher publisher = publisher(); |
49765 | 1019 |
publisher.enqueueUnordered(List.of(buf)); |
48083 | 1020 |
publisher.signalEnqueued(); |
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
1021 |
// mark preface sent. |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
1022 |
framesController.markPrefaceSent(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1023 |
Log.logTrace("PREFACE_BYTES sent"); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1024 |
Log.logTrace("Settings Frame sent"); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1025 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1026 |
// send a Window update for the receive buffer we are using |
50985
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1027 |
// minus the initial 64 K -1 specified in protocol: |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1028 |
// RFC 7540, Section 6.9.2: |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1029 |
// "[...] the connection flow-control window is set to the default |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1030 |
// initial window size until a WINDOW_UPDATE frame is received." |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1031 |
// |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1032 |
// Note that the default initial window size, not to be confused |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1033 |
// with the initial window size, is defined by RFC 7540 as |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1034 |
// 64K -1. |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1035 |
final int len = windowUpdater.initialWindowSize - DEFAULT_INITIAL_WINDOW_SIZE; |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1036 |
if (len != 0) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1037 |
if (Log.channel()) { |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1038 |
Log.logChannel("Sending initial connection window update frame: {0} ({1} - {2})", |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1039 |
len, windowUpdater.initialWindowSize, DEFAULT_INITIAL_WINDOW_SIZE); |
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1040 |
} |
48263
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
1041 |
windowUpdater.sendWindowUpdate(len); |
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
1042 |
} |
45530
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
1043 |
// there will be an ACK to the windows update - which should |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
1044 |
// cause any pending data stored before the preface was sent to be |
3c98842fddf7
8181430: HTTP/2 client might deadlock when receiving data during the initial handshake
dfuchs
parents:
44854
diff
changeset
|
1045 |
// flushed (see PrefaceController). |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1046 |
Log.logTrace("finished sending connection preface"); |
49765 | 1047 |
if (debug.on()) |
1048 |
debug.log("Triggering processing of buffered data" |
|
1049 |
+ " after sending connection preface"); |
|
48083 | 1050 |
subscriber.onNext(List.of(EMPTY_TRIGGER)); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1051 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1052 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1053 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1054 |
* Returns an existing Stream with given id, or null if doesn't exist |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1055 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1056 |
@SuppressWarnings("unchecked") |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1057 |
<T> Stream<T> getStream(int streamid) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1058 |
return (Stream<T>)streams.get(streamid); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1059 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1060 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1061 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1062 |
* Creates Stream with given id. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1063 |
*/ |
48083 | 1064 |
final <T> Stream<T> createStream(Exchange<T> exchange) { |
1065 |
Stream<T> stream = new Stream<>(this, exchange, windowController); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1066 |
return stream; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1067 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1068 |
|
49765 | 1069 |
<T> Stream.PushedStream<T> createPushStream(Stream<T> parent, Exchange<T> pushEx) { |
1070 |
PushGroup<T> pg = parent.exchange.getPushGroup(); |
|
48083 | 1071 |
return new Stream.PushedStream<>(pg, this, pushEx); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1072 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1073 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1074 |
<T> void putStream(Stream<T> stream, int streamid) { |
48083 | 1075 |
// increment the reference count on the HttpClientImpl |
1076 |
// to prevent the SelectorManager thread from exiting until |
|
1077 |
// the stream is closed. |
|
49765 | 1078 |
client().streamReference(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1079 |
streams.put(streamid, stream); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1080 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1081 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1082 |
/** |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1083 |
* Encode the headers into a List<ByteBuffer> and then create HEADERS |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1084 |
* and CONTINUATION frames from the list and return the List<Http2Frame>. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1085 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1086 |
private List<HeaderFrame> encodeHeaders(OutgoingHeaders<Stream<?>> frame) { |
48083 | 1087 |
List<ByteBuffer> buffers = encodeHeadersImpl( |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1088 |
getMaxSendFrameSize(), |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1089 |
frame.getAttachment().getRequestPseudoHeaders(), |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1090 |
frame.getUserHeaders(), |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1091 |
frame.getSystemHeaders()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1092 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1093 |
List<HeaderFrame> frames = new ArrayList<>(buffers.size()); |
48083 | 1094 |
Iterator<ByteBuffer> bufIterator = buffers.iterator(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1095 |
HeaderFrame oframe = new HeadersFrame(frame.streamid(), frame.getFlags(), bufIterator.next()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1096 |
frames.add(oframe); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1097 |
while(bufIterator.hasNext()) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1098 |
oframe = new ContinuationFrame(frame.streamid(), bufIterator.next()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1099 |
frames.add(oframe); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1100 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1101 |
oframe.setFlag(HeaderFrame.END_HEADERS); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1102 |
return frames; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1103 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1104 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1105 |
// Dedicated cache for headers encoding ByteBuffer. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1106 |
// There can be no concurrent access to this buffer as all access to this buffer |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1107 |
// and its content happen within a single critical code block section protected |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1108 |
// by the sendLock. / (see sendFrame()) |
48083 | 1109 |
// private final ByteBufferPool headerEncodingPool = new ByteBufferPool(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1110 |
|
48083 | 1111 |
private ByteBuffer getHeaderBuffer(int maxFrameSize) { |
1112 |
ByteBuffer buf = ByteBuffer.allocate(maxFrameSize); |
|
1113 |
buf.limit(maxFrameSize); |
|
1114 |
return buf; |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1115 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1116 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1117 |
/* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1118 |
* Encodes all the headers from the given HttpHeaders into the given List |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1119 |
* of buffers. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1120 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1121 |
* From https://tools.ietf.org/html/rfc7540#section-8.1.2 : |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1122 |
* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1123 |
* ...Just as in HTTP/1.x, header field names are strings of ASCII |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1124 |
* characters that are compared in a case-insensitive fashion. However, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1125 |
* header field names MUST be converted to lowercase prior to their |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1126 |
* encoding in HTTP/2... |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1127 |
*/ |
48083 | 1128 |
private List<ByteBuffer> encodeHeadersImpl(int maxFrameSize, HttpHeaders... headers) { |
1129 |
ByteBuffer buffer = getHeaderBuffer(maxFrameSize); |
|
1130 |
List<ByteBuffer> buffers = new ArrayList<>(); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1131 |
for(HttpHeaders header : headers) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1132 |
for (Map.Entry<String, List<String>> e : header.map().entrySet()) { |
49765 | 1133 |
String lKey = e.getKey().toLowerCase(Locale.US); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1134 |
List<String> values = e.getValue(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1135 |
for (String value : values) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1136 |
hpackOut.header(lKey, value); |
48083 | 1137 |
while (!hpackOut.encode(buffer)) { |
1138 |
buffer.flip(); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1139 |
buffers.add(buffer); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1140 |
buffer = getHeaderBuffer(maxFrameSize); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1141 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1142 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1143 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1144 |
} |
48083 | 1145 |
buffer.flip(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1146 |
buffers.add(buffer); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1147 |
return buffers; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1148 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1149 |
|
48083 | 1150 |
private List<ByteBuffer> encodeHeaders(OutgoingHeaders<Stream<?>> oh, Stream<?> stream) { |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1151 |
oh.streamid(stream.streamid); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1152 |
if (Log.headers()) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1153 |
StringBuilder sb = new StringBuilder("HEADERS FRAME (stream="); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1154 |
sb.append(stream.streamid).append(")\n"); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1155 |
Log.dumpHeaders(sb, " ", oh.getAttachment().getRequestPseudoHeaders()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1156 |
Log.dumpHeaders(sb, " ", oh.getSystemHeaders()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1157 |
Log.dumpHeaders(sb, " ", oh.getUserHeaders()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1158 |
Log.logHeaders(sb.toString()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1159 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1160 |
List<HeaderFrame> frames = encodeHeaders(oh); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1161 |
return encodeFrames(frames); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1162 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1163 |
|
48083 | 1164 |
private List<ByteBuffer> encodeFrames(List<HeaderFrame> frames) { |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1165 |
if (Log.frames()) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1166 |
frames.forEach(f -> Log.logFrames(f, "OUT")); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1167 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1168 |
return framesEncoder.encodeFrames(frames); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1169 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1170 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1171 |
private Stream<?> registerNewStream(OutgoingHeaders<Stream<?>> oh) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1172 |
Stream<?> stream = oh.getAttachment(); |
50985
cd41f34e548c
8206001: Enable TLS1.3 by default in Http Client
michaelm
parents:
50681
diff
changeset
|
1173 |
assert stream.streamid == 0; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1174 |
int streamid = nextstreamid; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1175 |
nextstreamid += 2; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1176 |
stream.registerStream(streamid); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1177 |
// set outgoing window here. This allows thread sending |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1178 |
// body to proceed. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1179 |
windowController.registerStream(streamid, getInitialSendWindowSize()); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1180 |
return stream; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1181 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1182 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1183 |
private final Object sendlock = new Object(); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1184 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1185 |
void sendFrame(Http2Frame frame) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1186 |
try { |
48083 | 1187 |
HttpPublisher publisher = publisher(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1188 |
synchronized (sendlock) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1189 |
if (frame instanceof OutgoingHeaders) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1190 |
@SuppressWarnings("unchecked") |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1191 |
OutgoingHeaders<Stream<?>> oh = (OutgoingHeaders<Stream<?>>) frame; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1192 |
Stream<?> stream = registerNewStream(oh); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1193 |
// provide protection from inserting unordered frames between Headers and Continuation |
48083 | 1194 |
publisher.enqueue(encodeHeaders(oh, stream)); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1195 |
} else { |
48083 | 1196 |
publisher.enqueue(encodeFrame(frame)); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1197 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1198 |
} |
48083 | 1199 |
publisher.signalEnqueued(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1200 |
} catch (IOException e) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1201 |
if (!closed) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1202 |
Log.logError(e); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1203 |
shutdown(e); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1204 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1205 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1206 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1207 |
|
48083 | 1208 |
private List<ByteBuffer> encodeFrame(Http2Frame frame) { |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1209 |
Log.logFrames(frame, "OUT"); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1210 |
return framesEncoder.encodeFrame(frame); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1211 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1212 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1213 |
void sendDataFrame(DataFrame frame) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1214 |
try { |
48083 | 1215 |
HttpPublisher publisher = publisher(); |
1216 |
publisher.enqueue(encodeFrame(frame)); |
|
1217 |
publisher.signalEnqueued(); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1218 |
} catch (IOException e) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1219 |
if (!closed) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1220 |
Log.logError(e); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1221 |
shutdown(e); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1222 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1223 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1224 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1225 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1226 |
/* |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1227 |
* Direct call of the method bypasses synchronization on "sendlock" and |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1228 |
* allowed only of control frames: WindowUpdateFrame, PingFrame and etc. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1229 |
* prohibited for such frames as DataFrame, HeadersFrame, ContinuationFrame. |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1230 |
*/ |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1231 |
void sendUnorderedFrame(Http2Frame frame) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1232 |
try { |
48083 | 1233 |
HttpPublisher publisher = publisher(); |
1234 |
publisher.enqueueUnordered(encodeFrame(frame)); |
|
1235 |
publisher.signalEnqueued(); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1236 |
} catch (IOException e) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1237 |
if (!closed) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1238 |
Log.logError(e); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1239 |
shutdown(e); |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1240 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1241 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1242 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1243 |
|
48083 | 1244 |
/** |
1245 |
* A simple tube subscriber for reading from the connection flow. |
|
1246 |
*/ |
|
1247 |
final class Http2TubeSubscriber implements TubeSubscriber { |
|
50681 | 1248 |
private volatile Flow.Subscription subscription; |
1249 |
private volatile boolean completed; |
|
1250 |
private volatile boolean dropped; |
|
1251 |
private volatile Throwable error; |
|
1252 |
private final ConcurrentLinkedQueue<ByteBuffer> queue |
|
48083 | 1253 |
= new ConcurrentLinkedQueue<>(); |
50681 | 1254 |
private final SequentialScheduler scheduler = |
48083 | 1255 |
SequentialScheduler.synchronizedScheduler(this::processQueue); |
50681 | 1256 |
private final HttpClientImpl client; |
1257 |
||
1258 |
Http2TubeSubscriber(HttpClientImpl client) { |
|
1259 |
this.client = Objects.requireNonNull(client); |
|
1260 |
} |
|
48083 | 1261 |
|
1262 |
final void processQueue() { |
|
1263 |
try { |
|
1264 |
while (!queue.isEmpty() && !scheduler.isStopped()) { |
|
1265 |
ByteBuffer buffer = queue.poll(); |
|
49765 | 1266 |
if (debug.on()) |
1267 |
debug.log("sending %d to Http2Connection.asyncReceive", |
|
1268 |
buffer.remaining()); |
|
48083 | 1269 |
asyncReceive(buffer); |
1270 |
} |
|
1271 |
} catch (Throwable t) { |
|
1272 |
Throwable x = error; |
|
1273 |
if (x == null) error = t; |
|
1274 |
} finally { |
|
1275 |
Throwable x = error; |
|
1276 |
if (x != null) { |
|
49765 | 1277 |
if (debug.on()) debug.log("Stopping scheduler", x); |
48083 | 1278 |
scheduler.stop(); |
1279 |
Http2Connection.this.shutdown(x); |
|
1280 |
} |
|
1281 |
} |
|
1282 |
} |
|
1283 |
||
50681 | 1284 |
private final void runOrSchedule() { |
1285 |
if (client.isSelectorThread()) { |
|
1286 |
scheduler.runOrSchedule(client.theExecutor()); |
|
1287 |
} else scheduler.runOrSchedule(); |
|
1288 |
} |
|
1289 |
||
49765 | 1290 |
@Override |
48083 | 1291 |
public void onSubscribe(Flow.Subscription subscription) { |
1292 |
// supports being called multiple time. |
|
1293 |
// doesn't cancel the previous subscription, since that is |
|
1294 |
// most probably the same as the new subscription. |
|
1295 |
assert this.subscription == null || dropped == false; |
|
1296 |
this.subscription = subscription; |
|
1297 |
dropped = false; |
|
1298 |
// TODO FIXME: request(1) should be done by the delegate. |
|
1299 |
if (!completed) { |
|
49765 | 1300 |
if (debug.on()) |
1301 |
debug.log("onSubscribe: requesting Long.MAX_VALUE for reading"); |
|
48083 | 1302 |
subscription.request(Long.MAX_VALUE); |
1303 |
} else { |
|
49765 | 1304 |
if (debug.on()) debug.log("onSubscribe: already completed"); |
48083 | 1305 |
} |
1306 |
} |
|
1307 |
||
1308 |
@Override |
|
1309 |
public void onNext(List<ByteBuffer> item) { |
|
49765 | 1310 |
if (debug.on()) debug.log(() -> "onNext: got " + Utils.remaining(item) |
48083 | 1311 |
+ " bytes in " + item.size() + " buffers"); |
1312 |
queue.addAll(item); |
|
50681 | 1313 |
runOrSchedule(); |
48083 | 1314 |
} |
1315 |
||
1316 |
@Override |
|
1317 |
public void onError(Throwable throwable) { |
|
49765 | 1318 |
if (debug.on()) debug.log(() -> "onError: " + throwable); |
48083 | 1319 |
error = throwable; |
1320 |
completed = true; |
|
50681 | 1321 |
runOrSchedule(); |
48083 | 1322 |
} |
1323 |
||
1324 |
@Override |
|
1325 |
public void onComplete() { |
|
50681 | 1326 |
String msg = isActive() |
1327 |
? "EOF reached while reading" |
|
1328 |
: "Idle connection closed by HTTP/2 peer"; |
|
1329 |
if (debug.on()) debug.log(msg); |
|
1330 |
error = new EOFException(msg); |
|
48083 | 1331 |
completed = true; |
50681 | 1332 |
runOrSchedule(); |
48083 | 1333 |
} |
1334 |
||
49765 | 1335 |
@Override |
48083 | 1336 |
public void dropSubscription() { |
49765 | 1337 |
if (debug.on()) debug.log("dropSubscription"); |
48083 | 1338 |
// we could probably set subscription to null here... |
1339 |
// then we might not need the 'dropped' boolean? |
|
1340 |
dropped = true; |
|
1341 |
} |
|
1342 |
} |
|
1343 |
||
50681 | 1344 |
synchronized boolean isActive() { |
1345 |
return numReservedClientStreams > 0 || numReservedServerStreams > 0; |
|
1346 |
} |
|
1347 |
||
48083 | 1348 |
@Override |
1349 |
public final String toString() { |
|
1350 |
return dbgString(); |
|
1351 |
} |
|
1352 |
||
1353 |
final String dbgString() { |
|
1354 |
return "Http2Connection(" |
|
1355 |
+ connection.getConnectionFlow() + ")"; |
|
1356 |
} |
|
1357 |
||
49765 | 1358 |
static class HeaderDecoder extends ValidatingHeadersConsumer { |
48083 | 1359 |
|
50681 | 1360 |
HttpHeadersBuilder headersBuilder; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1361 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1362 |
HeaderDecoder() { |
50681 | 1363 |
this.headersBuilder = new HttpHeadersBuilder(); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1364 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1365 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1366 |
@Override |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1367 |
public void onDecoded(CharSequence name, CharSequence value) { |
49765 | 1368 |
String n = name.toString(); |
1369 |
String v = value.toString(); |
|
1370 |
super.onDecoded(n, v); |
|
50681 | 1371 |
headersBuilder.addHeader(n, v); |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1372 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1373 |
|
50681 | 1374 |
HttpHeaders headers() { |
1375 |
return headersBuilder.build(); |
|
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1376 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1377 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1378 |
|
49765 | 1379 |
/* |
1380 |
* Checks RFC 7540 rules (relaxed) compliance regarding pseudo-headers. |
|
1381 |
*/ |
|
1382 |
static class ValidatingHeadersConsumer implements DecodingCallback { |
|
1383 |
||
1384 |
private static final Set<String> PSEUDO_HEADERS = |
|
1385 |
Set.of(":authority", ":method", ":path", ":scheme", ":status"); |
|
1386 |
||
1387 |
/** Used to check that if there are pseudo-headers, they go first */ |
|
1388 |
private boolean pseudoHeadersEnded; |
|
1389 |
||
1390 |
/** |
|
1391 |
* Called when END_HEADERS was received. This consumer may be invoked |
|
1392 |
* again after reset() is called, but for a whole new set of headers. |
|
1393 |
*/ |
|
1394 |
void reset() { |
|
1395 |
pseudoHeadersEnded = false; |
|
1396 |
} |
|
1397 |
||
1398 |
@Override |
|
1399 |
public void onDecoded(CharSequence name, CharSequence value) |
|
1400 |
throws UncheckedIOException |
|
1401 |
{ |
|
1402 |
String n = name.toString(); |
|
1403 |
if (n.startsWith(":")) { |
|
1404 |
if (pseudoHeadersEnded) { |
|
1405 |
throw newException("Unexpected pseudo-header '%s'", n); |
|
1406 |
} else if (!PSEUDO_HEADERS.contains(n)) { |
|
1407 |
throw newException("Unknown pseudo-header '%s'", n); |
|
1408 |
} |
|
1409 |
} else { |
|
1410 |
pseudoHeadersEnded = true; |
|
1411 |
if (!Utils.isValidName(n)) { |
|
1412 |
throw newException("Bad header name '%s'", n); |
|
1413 |
} |
|
1414 |
} |
|
1415 |
String v = value.toString(); |
|
1416 |
if (!Utils.isValidValue(v)) { |
|
1417 |
throw newException("Bad header value '%s'", v); |
|
1418 |
} |
|
1419 |
} |
|
1420 |
||
1421 |
private UncheckedIOException newException(String message, String header) |
|
1422 |
{ |
|
1423 |
return new UncheckedIOException( |
|
1424 |
new IOException(String.format(message, header))); |
|
1425 |
} |
|
1426 |
} |
|
1427 |
||
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1428 |
static final class ConnectionWindowUpdateSender extends WindowUpdateSender { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1429 |
|
48263
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
1430 |
final int initialWindowSize; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1431 |
public ConnectionWindowUpdateSender(Http2Connection connection, |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1432 |
int initialWindowSize) { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1433 |
super(connection, initialWindowSize); |
48263
a559b7cd1dea
8193370: Provide more user friendly defaults for HTTP/2 client settings
dfuchs
parents:
48083
diff
changeset
|
1434 |
this.initialWindowSize = initialWindowSize; |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1435 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1436 |
|
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1437 |
@Override |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1438 |
int getStreamId() { |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1439 |
return 0; |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1440 |
} |
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1441 |
} |
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1442 |
|
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1443 |
/** |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1444 |
* Thrown when https handshake negotiates http/1.1 alpn instead of h2 |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1445 |
*/ |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1446 |
static final class ALPNException extends IOException { |
49765 | 1447 |
private static final long serialVersionUID = 0L; |
1448 |
final transient AbstractAsyncSSLConnection connection; |
|
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1449 |
|
46157 | 1450 |
ALPNException(String msg, AbstractAsyncSSLConnection connection) { |
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1451 |
super(msg); |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1452 |
this.connection = connection; |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1453 |
} |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1454 |
|
46157 | 1455 |
AbstractAsyncSSLConnection getConnection() { |
44854
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1456 |
return connection; |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1457 |
} |
5a486e0acd29
8175814: Update default HttpClient protocol version and optional request version
michaelm
parents:
43984
diff
changeset
|
1458 |
} |
42460
7133f144981a
8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff
changeset
|
1459 |
} |