src/java.net.http/share/classes/jdk/internal/net/http/Stream.java
author dfuchs
Fri, 04 May 2018 17:02:19 +0100
branchhttp-client-branch
changeset 56531 15ff86a732ea
parent 56514 6ef5ca8283a4
child 56616 5d2446adafaf
permissions -rw-r--r--
http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
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: 48083
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
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    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
56514
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
    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
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    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.URI;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    32
import java.nio.ByteBuffer;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    33
import java.util.ArrayList;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    34
import java.util.Collections;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    35
import java.util.List;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    36
import java.util.concurrent.CompletableFuture;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    37
import java.util.concurrent.ConcurrentLinkedDeque;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    38
import java.util.concurrent.ConcurrentLinkedQueue;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    39
import java.util.concurrent.Executor;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    40
import java.util.concurrent.Flow;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    41
import java.util.concurrent.Flow.Subscription;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    42
import java.util.concurrent.atomic.AtomicReference;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    43
import java.util.function.BiPredicate;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    44
import java.net.http.HttpClient;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    45
import java.net.http.HttpHeaders;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    46
import java.net.http.HttpRequest;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    47
import java.net.http.HttpResponse;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    48
import java.net.http.HttpResponse.BodySubscriber;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    49
import jdk.internal.net.http.common.*;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    50
import jdk.internal.net.http.frame.*;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    51
import jdk.internal.net.http.hpack.DecodingCallback;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    52
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    53
/**
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    54
 * Http/2 Stream handling.
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    55
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    56
 * REQUESTS
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    57
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    58
 * sendHeadersOnly() -- assembles HEADERS frame and puts on connection outbound Q
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    59
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    60
 * sendRequest() -- sendHeadersOnly() + sendBody()
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    61
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    62
 * sendBodyAsync() -- calls sendBody() in an executor thread.
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    63
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    64
 * sendHeadersAsync() -- calls sendHeadersOnly() which does not block
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    65
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    66
 * sendRequestAsync() -- calls sendRequest() in an executor thread
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    67
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    68
 * RESPONSES
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    69
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    70
 * Multiple responses can be received per request. Responses are queued up on
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    71
 * a LinkedList of CF<HttpResponse> and the the first one on the list is completed
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    72
 * with the next response
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    73
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    74
 * getResponseAsync() -- queries list of response CFs and returns first one
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    75
 *               if one exists. Otherwise, creates one and adds it to list
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    76
 *               and returns it. Completion is achieved through the
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    77
 *               incoming() upcall from connection reader thread.
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    78
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    79
 * getResponse() -- calls getResponseAsync() and waits for CF to complete
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    80
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    81
 * responseBodyAsync() -- calls responseBody() in an executor thread.
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    82
 *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    83
 * incoming() -- entry point called from connection reader thread. Frames are
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    84
 *               either handled immediately without blocking or for data frames
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    85
 *               placed on the stream's inputQ which is consumed by the stream's
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    86
 *               reader thread.
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
 * PushedStream sub class
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    89
 * ======================
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    90
 * Sending side methods are not used because the request comes from a PUSH_PROMISE
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    91
 * frame sent by the server. When a PUSH_PROMISE is received the PushedStream
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    92
 * is created. PushedStream does not use responseCF list as there can be only
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    93
 * one response. The CF is created when the object created and when the response
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    94
 * HEADERS frame is received the object is completed.
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
class Stream<T> extends ExchangeImpl<T> {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
    97
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
    98
    final Logger debug = Utils.getDebugLogger(this::dbgString, Utils.DEBUG);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    99
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   100
    final ConcurrentLinkedQueue<Http2Frame> inputQ = new ConcurrentLinkedQueue<>();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   101
    final SequentialScheduler sched =
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   102
            SequentialScheduler.synchronizedScheduler(this::schedule);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   103
    final SubscriptionBase userSubscription =
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   104
            new SubscriptionBase(sched, this::cancel, this::onSubscriptionError);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   105
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
     * This stream's identifier. Assigned lazily by the HTTP2Connection before
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   108
     * the stream's first frame is sent.
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   109
     */
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   110
    protected volatile int streamid;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   111
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   112
    long requestContentLen;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   113
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   114
    final Http2Connection connection;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   115
    final HttpRequestImpl request;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   116
    final HeadersConsumer rspHeadersConsumer;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   117
    final HttpHeadersImpl responseHeaders;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   118
    final HttpHeadersImpl requestPseudoHeaders;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   119
    volatile HttpResponse.BodySubscriber<T> responseSubscriber;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   120
    final HttpRequest.BodyPublisher requestPublisher;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   121
    volatile RequestSubscriber requestSubscriber;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   122
    volatile int responseCode;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   123
    volatile Response response;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   124
    // The exception with which this stream was canceled.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   125
    private final AtomicReference<Throwable> errorRef = new AtomicReference<>();
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   126
    final CompletableFuture<Void> requestBodyCF = new MinimalFuture<>();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   127
    volatile CompletableFuture<T> responseBodyCF;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   128
    volatile HttpResponse.BodySubscriber<T> pendingResponseSubscriber;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   129
    volatile boolean stopRequested;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   130
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   131
    /** True if END_STREAM has been seen in a frame received on this stream. */
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   132
    private volatile boolean remotelyClosed;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   133
    private volatile boolean closed;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   134
    private volatile boolean endStreamSent;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   135
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   136
    // state flags
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   137
    private boolean requestSent, responseReceived;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   138
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   139
    /**
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   140
     * A reference to this Stream's connection Send Window controller. The
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   141
     * stream MUST acquire the appropriate amount of Send Window before
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   142
     * sending any data. Will be null for PushStreams, as they cannot send data.
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
    private final WindowController windowController;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   145
    private final WindowUpdateSender windowUpdater;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   146
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   147
    @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   148
    HttpConnection connection() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   149
        return connection.connection;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   150
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   151
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   152
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   153
     * Invoked either from incoming() -> {receiveDataFrame() or receiveResetFrame() }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   154
     * of after user subscription window has re-opened, from SubscriptionBase.request()
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   155
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   156
    private void schedule() {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   157
        boolean onCompleteCalled = false;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   158
        HttpResponse.BodySubscriber<T> subscriber = responseSubscriber;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   159
        try {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   160
            if (subscriber == null) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   161
                subscriber = responseSubscriber = pendingResponseSubscriber;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   162
                if (subscriber == null) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   163
                    // can't process anything yet
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   164
                    return;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   165
                } else {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   166
                    if (debug.on()) debug.log("subscribing user subscriber");
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   167
                    subscriber.onSubscribe(userSubscription);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   168
                }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   169
            }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   170
            while (!inputQ.isEmpty()) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   171
                Http2Frame frame = inputQ.peek();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   172
                if (frame instanceof ResetFrame) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   173
                    inputQ.remove();
56514
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   174
                    handleReset((ResetFrame)frame, subscriber);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   175
                    return;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   176
                }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   177
                DataFrame df = (DataFrame)frame;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   178
                boolean finished = df.getFlag(DataFrame.END_STREAM);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   179
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   180
                List<ByteBuffer> buffers = df.getData();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   181
                List<ByteBuffer> dsts = Collections.unmodifiableList(buffers);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   182
                int size = Utils.remaining(dsts, Integer.MAX_VALUE);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   183
                if (size == 0 && finished) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   184
                    inputQ.remove();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   185
                    Log.logTrace("responseSubscriber.onComplete");
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   186
                    if (debug.on()) debug.log("incoming: onComplete");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   187
                    sched.stop();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   188
                    subscriber.onComplete();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   189
                    onCompleteCalled = true;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   190
                    setEndStreamReceived();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   191
                    return;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   192
                } else if (userSubscription.tryDecrement()) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   193
                    inputQ.remove();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   194
                    Log.logTrace("responseSubscriber.onNext {0}", size);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   195
                    if (debug.on()) debug.log("incoming: onNext(%d)", size);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   196
                    subscriber.onNext(dsts);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   197
                    if (consumed(df)) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   198
                        Log.logTrace("responseSubscriber.onComplete");
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   199
                        if (debug.on()) debug.log("incoming: onComplete");
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   200
                        sched.stop();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   201
                        subscriber.onComplete();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   202
                        onCompleteCalled = true;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   203
                        setEndStreamReceived();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   204
                        return;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   205
                    }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   206
                } else {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   207
                    if (stopRequested) break;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   208
                    return;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   209
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   210
            }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   211
        } catch (Throwable throwable) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   212
            errorRef.compareAndSet(null, throwable);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   213
        }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   214
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   215
        Throwable t = errorRef.get();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   216
        if (t != null) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   217
            sched.stop();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   218
            try {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   219
                if (!onCompleteCalled) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   220
                    if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   221
                        debug.log("calling subscriber.onError: %s", (Object)t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   222
                    subscriber.onError(t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   223
                } else {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   224
                    if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   225
                        debug.log("already completed: dropping error %s", (Object)t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   226
                }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   227
            } catch (Throwable x) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   228
                Log.logError("Subscriber::onError threw exception: {0}", (Object)t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   229
            } finally {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   230
                cancelImpl(t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   231
            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   232
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   233
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   234
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   235
    // Callback invoked after the Response BodySubscriber has consumed the
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   236
    // buffers contained in a DataFrame.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   237
    // Returns true if END_STREAM is reached, false otherwise.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   238
    private boolean consumed(DataFrame df) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   239
        // RFC 7540 6.1:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   240
        // The entire DATA frame payload is included in flow control,
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   241
        // including the Pad Length and Padding fields if present
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   242
        int len = df.payloadLength();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   243
        connection.windowUpdater.update(len);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   244
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   245
        if (!df.getFlag(DataFrame.END_STREAM)) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   246
            // Don't send window update on a stream which is
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   247
            // closed or half closed.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   248
            windowUpdater.update(len);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   249
            return false; // more data coming
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   250
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   251
        return true; // end of stream
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   252
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   253
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   254
    @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   255
    CompletableFuture<T> readBodyAsync(HttpResponse.BodyHandler<T> handler,
43999
4cc44dd9f14f 8164625: Pooled HttpConnection should be removed during close
prappo
parents: 43984
diff changeset
   256
                                       boolean returnConnectionToPool,
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   257
                                       Executor executor)
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   258
    {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   259
        try {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   260
            Log.logTrace("Reading body on stream {0}", streamid);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   261
            BodySubscriber<T> bodySubscriber = handler.apply(new ResponseInfoImpl(response));
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   262
            CompletableFuture<T> cf = receiveData(bodySubscriber, executor);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   263
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   264
            PushGroup<?> pg = exchange.getPushGroup();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   265
            if (pg != null) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   266
                // if an error occurs make sure it is recorded in the PushGroup
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   267
                cf = cf.whenComplete((t, e) -> pg.pushError(e));
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   268
            }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   269
            return cf;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   270
        } catch (Throwable t) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   271
            // may be thrown by handler.apply
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   272
            cancelImpl(t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   273
            return MinimalFuture.failedFuture(t);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   274
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   275
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   276
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   277
    @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   278
    public String toString() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   279
        StringBuilder sb = new StringBuilder();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   280
        sb.append("streamid: ")
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   281
                .append(streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   282
        return sb.toString();
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
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   285
    private void receiveDataFrame(DataFrame df) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   286
        inputQ.add(df);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   287
        sched.runOrSchedule();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   288
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   289
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   290
    /** Handles a RESET frame. RESET is always handled inline in the queue. */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   291
    private void receiveResetFrame(ResetFrame frame) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   292
        inputQ.add(frame);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   293
        sched.runOrSchedule();
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   294
    }
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   295
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   296
    // pushes entire response body into response subscriber
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   297
    // blocking when required by local or remote flow control
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   298
    CompletableFuture<T> receiveData(BodySubscriber<T> bodySubscriber, Executor executor) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   299
        responseBodyCF = new MinimalFuture<>();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   300
        // We want to allow the subscriber's getBody() method to block so it
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   301
        // can work with InputStreams. So, we offload execution.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   302
        executor.execute(() -> {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   303
            try {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   304
                bodySubscriber.getBody().whenComplete((T body, Throwable t) -> {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   305
                    if (t == null)
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   306
                        responseBodyCF.complete(body);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   307
                    else
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   308
                        responseBodyCF.completeExceptionally(t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   309
                });
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   310
            } catch(Throwable t) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   311
                cancelImpl(t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   312
            }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   313
        });
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   314
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   315
        if (isCanceled()) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   316
            Throwable t = getCancelCause();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   317
            responseBodyCF.completeExceptionally(t);
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   318
        } else {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   319
            pendingResponseSubscriber = bodySubscriber;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   320
            sched.runOrSchedule(); // in case data waiting already to be processed
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   321
        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   322
        return responseBodyCF;
42460
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
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   325
    @Override
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   326
    CompletableFuture<ExchangeImpl<T>> sendBodyAsync() {
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   327
        return sendBodyImpl().thenApply( v -> this);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   328
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   329
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   330
    @SuppressWarnings("unchecked")
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   331
    Stream(Http2Connection connection,
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   332
           Exchange<T> e,
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   333
           WindowController windowController)
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   334
    {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   335
        super(e);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   336
        this.connection = connection;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   337
        this.windowController = windowController;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   338
        this.request = e.request();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   339
        this.requestPublisher = request.requestPublisher;  // may be null
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   340
        responseHeaders = new HttpHeadersImpl();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   341
        rspHeadersConsumer = new HeadersConsumer();
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   342
        this.requestPseudoHeaders = new HttpHeadersImpl();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   343
        // NEW
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   344
        this.windowUpdater = new StreamWindowUpdateSender(connection);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   345
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   346
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   347
    /**
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   348
     * Entry point from Http2Connection reader thread.
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   349
     *
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   350
     * Data frames will be removed by response body thread.
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   351
     */
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   352
    void incoming(Http2Frame frame) throws IOException {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   353
        if (debug.on()) debug.log("incoming: %s", frame);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   354
        if ((frame instanceof HeaderFrame)) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   355
            HeaderFrame hframe = (HeaderFrame)frame;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   356
            if (hframe.endHeaders()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   357
                Log.logTrace("handling response (streamid={0})", streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   358
                handleResponse();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   359
                if (hframe.getFlag(HeaderFrame.END_STREAM)) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   360
                    receiveDataFrame(new DataFrame(streamid, DataFrame.END_STREAM, List.of()));
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   361
                }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   362
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   363
        } else if (frame instanceof DataFrame) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   364
            receiveDataFrame((DataFrame)frame);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   365
        } else {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   366
            otherFrame(frame);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   367
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   368
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   369
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   370
    void otherFrame(Http2Frame frame) throws IOException {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   371
        switch (frame.type()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   372
            case WindowUpdateFrame.TYPE:
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   373
                incoming_windowUpdate((WindowUpdateFrame) frame);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   374
                break;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   375
            case ResetFrame.TYPE:
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   376
                incoming_reset((ResetFrame) frame);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   377
                break;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   378
            case PriorityFrame.TYPE:
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   379
                incoming_priority((PriorityFrame) frame);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   380
                break;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   381
            default:
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   382
                String msg = "Unexpected frame: " + frame.toString();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   383
                throw new IOException(msg);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   384
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   385
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   386
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   387
    // The Hpack decoder decodes into one of these consumers of name,value pairs
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   388
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   389
    DecodingCallback rspHeadersConsumer() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   390
        return rspHeadersConsumer;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   391
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   392
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   393
    protected void handleResponse() throws IOException {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   394
        responseCode = (int)responseHeaders
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   395
                .firstValueAsLong(":status")
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   396
                .orElseThrow(() -> new IOException("no statuscode in response"));
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
        response = new Response(
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   399
                request, exchange, responseHeaders, connection(),
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   400
                responseCode, HttpClient.Version.HTTP_2);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   401
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   402
        /* TODO: review if needs to be removed
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   403
           the value is not used, but in case `content-length` doesn't parse as
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   404
           long, there will be NumberFormatException. If left as is, make sure
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   405
           code up the stack handles NFE correctly. */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   406
        responseHeaders.firstValueAsLong("content-length");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   407
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   408
        if (Log.headers()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   409
            StringBuilder sb = new StringBuilder("RESPONSE HEADERS:\n");
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   410
            Log.dumpHeaders(sb, "    ", responseHeaders);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   411
            Log.logHeaders(sb.toString());
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   412
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   413
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   414
        // this will clear the response headers
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   415
        rspHeadersConsumer.reset();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   416
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   417
        completeResponse(response);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   418
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   419
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   420
    void incoming_reset(ResetFrame frame) {
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   421
        Log.logTrace("Received RST_STREAM on stream {0}", streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   422
        if (endStreamReceived()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   423
            Log.logTrace("Ignoring RST_STREAM frame received on remotely closed stream {0}", streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   424
        } else if (closed) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   425
            Log.logTrace("Ignoring RST_STREAM frame received on closed stream {0}", streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   426
        } else {
56514
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   427
            Flow.Subscriber<?> subscriber =
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   428
                    responseSubscriber == null ? pendingResponseSubscriber : responseSubscriber;
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   429
            if (response == null && subscriber == null) {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   430
                // we haven't receive the headers yet, and won't receive any!
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   431
                // handle reset now.
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   432
                handleReset(frame, subscriber);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   433
            } else {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   434
                // put it in the input queue in order to read all
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   435
                // pending data frames first. Indeed, a server may send
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   436
                // RST_STREAM after sending END_STREAM, in which case we should
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   437
                // ignore it. However, we won't know if we have received END_STREAM
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   438
                // or not until all pending data frames are read.
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   439
                receiveResetFrame(frame);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   440
                // RST_STREAM was pushed to the queue. It will be handled by
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   441
                // asyncReceive after all pending data frames have been
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   442
                // processed.
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   443
                Log.logTrace("RST_STREAM pushed in queue for stream {0}", streamid);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   444
            }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   445
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   446
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   447
56514
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   448
    void handleReset(ResetFrame frame, Flow.Subscriber<?> subscriber) {
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   449
        Log.logTrace("Handling RST_STREAM on stream {0}", streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   450
        if (!closed) {
56514
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   451
            synchronized (this) {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   452
                if (closed) {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   453
                    if (debug.on()) debug.log("Stream already closed: ignoring RESET");
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   454
                    return;
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   455
                }
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   456
                closed = true;
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   457
            }
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   458
            try {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   459
                int error = frame.getErrorCode();
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   460
                IOException e = new IOException("Received RST_STREAM: "
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   461
                        + ErrorFrame.stringForCode(error));
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   462
                if (errorRef.compareAndSet(null, e)) {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   463
                    if (subscriber != null) {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   464
                        subscriber.onError(e);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   465
                    }
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   466
                }
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   467
                completeResponseExceptionally(e);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   468
                if (!requestBodyCF.isDone()) {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   469
                    requestBodyCF.completeExceptionally(errorRef.get()); // we may be sending the body..
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   470
                }
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   471
                if (responseBodyCF != null) {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   472
                    responseBodyCF.completeExceptionally(errorRef.get());
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   473
                }
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   474
            } finally {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   475
                connection.closeStream(streamid);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
   476
            }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   477
        } else {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   478
            Log.logTrace("Ignoring RST_STREAM frame received on closed stream {0}", streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   479
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   480
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   481
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   482
    void incoming_priority(PriorityFrame frame) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   483
        // TODO: implement priority
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   484
        throw new UnsupportedOperationException("Not implemented");
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   485
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   486
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   487
    private void incoming_windowUpdate(WindowUpdateFrame frame)
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   488
        throws IOException
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   489
    {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   490
        int amount = frame.getUpdate();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   491
        if (amount <= 0) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   492
            Log.logTrace("Resetting stream: {0} %d, Window Update amount: %d\n",
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   493
                         streamid, streamid, amount);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   494
            connection.resetStream(streamid, ResetFrame.FLOW_CONTROL_ERROR);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   495
        } else {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   496
            assert streamid != 0;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   497
            boolean success = windowController.increaseStreamWindow(amount, streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   498
            if (!success) {  // overflow
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   499
                connection.resetStream(streamid, ResetFrame.FLOW_CONTROL_ERROR);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   500
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   501
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   502
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   503
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   504
    void incoming_pushPromise(HttpRequestImpl pushRequest,
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   505
                              PushedStream<T> pushStream)
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   506
        throws IOException
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   507
    {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   508
        if (Log.requests()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   509
            Log.logRequest("PUSH_PROMISE: " + pushRequest.toString());
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   510
        }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   511
        PushGroup<T> pushGroup = exchange.getPushGroup();
48523
b95b08f3e1a8 8194883: Unhandleable Push Promises should be cancelled
chegar
parents: 48083
diff changeset
   512
        if (pushGroup == null) {
b95b08f3e1a8 8194883: Unhandleable Push Promises should be cancelled
chegar
parents: 48083
diff changeset
   513
            Log.logTrace("Rejecting push promise stream " + streamid);
b95b08f3e1a8 8194883: Unhandleable Push Promises should be cancelled
chegar
parents: 48083
diff changeset
   514
            connection.resetStream(pushStream.streamid, ResetFrame.REFUSED_STREAM);
b95b08f3e1a8 8194883: Unhandleable Push Promises should be cancelled
chegar
parents: 48083
diff changeset
   515
            pushStream.close();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   516
            return;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   517
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   518
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   519
        PushGroup.Acceptor<T> acceptor = null;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   520
        boolean accepted = false;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   521
        try {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   522
            acceptor = pushGroup.acceptPushRequest(pushRequest);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   523
            accepted = acceptor.accepted();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   524
        } catch (Throwable t) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   525
            if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   526
                debug.log("PushPromiseHandler::applyPushPromise threw exception %s",
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   527
                          (Object)t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   528
        }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   529
        if (!accepted) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   530
            // cancel / reject
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   531
            IOException ex = new IOException("Stream " + streamid + " cancelled by users handler");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   532
            if (Log.trace()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   533
                Log.logTrace("No body subscriber for {0}: {1}", pushRequest,
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   534
                        ex.getMessage());
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   535
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   536
            pushStream.cancelImpl(ex);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   537
            return;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   538
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   539
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   540
        assert accepted && acceptor != null;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   541
        CompletableFuture<HttpResponse<T>> pushResponseCF = acceptor.cf();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   542
        HttpResponse.BodyHandler<T> pushHandler = acceptor.bodyHandler();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   543
        assert pushHandler != null;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   544
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   545
        pushStream.requestSent();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   546
        pushStream.setPushHandler(pushHandler);  // TODO: could wrap the handler to throw on acceptPushPromise ?
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   547
        // setup housekeeping for when the push is received
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   548
        // TODO: deal with ignoring of CF anti-pattern
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   549
        CompletableFuture<HttpResponse<T>> cf = pushStream.responseCF();
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   550
        cf.whenComplete((HttpResponse<T> resp, Throwable t) -> {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   551
            t = Utils.getCompletionCause(t);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   552
            if (Log.trace()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   553
                Log.logTrace("Push completed on stream {0} for {1}{2}",
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   554
                             pushStream.streamid, resp,
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   555
                             ((t==null) ? "": " with exception " + t));
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   556
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   557
            if (t != null) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   558
                pushGroup.pushError(t);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   559
                pushResponseCF.completeExceptionally(t);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   560
            } else {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   561
                pushResponseCF.complete(resp);
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
            pushGroup.pushCompleted();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   564
        });
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   565
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   566
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   567
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   568
    private OutgoingHeaders<Stream<T>> headerFrame(long contentLength) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   569
        HttpHeadersImpl h = request.getSystemHeaders();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   570
        if (contentLength > 0) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   571
            h.setHeader("content-length", Long.toString(contentLength));
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   572
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   573
        setPseudoHeaderFields();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   574
        HttpHeaders sysh = filter(h);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   575
        HttpHeaders userh = filter(request.getUserHeaders());
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   576
        OutgoingHeaders<Stream<T>> f = new OutgoingHeaders<>(sysh, userh, this);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   577
        if (contentLength == 0) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   578
            f.setFlag(HeadersFrame.END_STREAM);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   579
            endStreamSent = true;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   580
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   581
        return f;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   582
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   583
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   584
    private boolean hasProxyAuthorization(HttpHeaders headers) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   585
        return headers.firstValue("proxy-authorization")
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   586
                      .isPresent();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   587
    }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   588
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   589
    // Determines whether we need to build a new HttpHeader object.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   590
    //
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   591
    // Ideally we should pass the filter to OutgoingHeaders refactor the
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   592
    // code that creates the HeaderFrame to honor the filter.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   593
    // We're not there yet - so depending on the filter we need to
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   594
    // apply and the content of the header we will try to determine
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   595
    //  whether anything might need to be filtered.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   596
    // If nothing needs filtering then we can just use the
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   597
    // original headers.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   598
    private boolean needsFiltering(HttpHeaders headers,
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   599
                                   BiPredicate<String, List<String>> filter) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   600
        if (filter == Utils.PROXY_TUNNEL_FILTER || filter == Utils.PROXY_FILTER) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   601
            // we're either connecting or proxying
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   602
            // slight optimization: we only need to filter out
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   603
            // disabled schemes, so if there are none just
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   604
            // pass through.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   605
            return Utils.proxyHasDisabledSchemes(filter == Utils.PROXY_TUNNEL_FILTER)
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   606
                    && hasProxyAuthorization(headers);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   607
        } else {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   608
            // we're talking to a server, either directly or through
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   609
            // a tunnel.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   610
            // Slight optimization: we only need to filter out
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   611
            // proxy authorization headers, so if there are none just
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   612
            // pass through.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   613
            return hasProxyAuthorization(headers);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   614
        }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   615
    }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   616
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   617
    private HttpHeaders filter(HttpHeaders headers) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   618
        HttpConnection conn = connection();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   619
        BiPredicate<String, List<String>> filter =
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   620
                conn.headerFilter(request);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   621
        if (needsFiltering(headers, filter)) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   622
            return ImmutableHeaders.of(headers.map(), filter);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   623
        }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   624
        return headers;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   625
    }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   626
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   627
    private void setPseudoHeaderFields() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   628
        HttpHeadersImpl hdrs = requestPseudoHeaders;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   629
        String method = request.method();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   630
        hdrs.setHeader(":method", method);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   631
        URI uri = request.uri();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   632
        hdrs.setHeader(":scheme", uri.getScheme());
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   633
        // TODO: userinfo deprecated. Needs to be removed
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   634
        hdrs.setHeader(":authority", uri.getAuthority());
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   635
        // TODO: ensure header names beginning with : not in user headers
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   636
        String query = uri.getRawQuery();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   637
        String path = uri.getRawPath();
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   638
        if (path == null || path.isEmpty()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   639
            if (method.equalsIgnoreCase("OPTIONS")) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   640
                path = "*";
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   641
            } else {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   642
                path = "/";
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   643
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   644
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   645
        if (query != null) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   646
            path += "?" + query;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   647
        }
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   648
        hdrs.setHeader(":path", Utils.encode(path));
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   649
    }
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
    HttpHeadersImpl getRequestPseudoHeaders() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   652
        return requestPseudoHeaders;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   653
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   654
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   655
    /** Sets endStreamReceived. Should be called only once. */
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   656
    void setEndStreamReceived() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   657
        assert remotelyClosed == false: "Unexpected endStream already set";
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   658
        remotelyClosed = true;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   659
        responseReceived();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   660
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   661
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   662
    /** Tells whether, or not, the END_STREAM Flag has been seen in any frame
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   663
     *  received on this stream. */
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   664
    private boolean endStreamReceived() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   665
        return remotelyClosed;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   666
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   667
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   668
    @Override
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   669
    CompletableFuture<ExchangeImpl<T>> sendHeadersAsync() {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   670
        if (debug.on()) debug.log("sendHeadersOnly()");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   671
        if (Log.requests() && request != null) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   672
            Log.logRequest(request.toString());
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   673
        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   674
        if (requestPublisher != null) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   675
            requestContentLen = requestPublisher.contentLength();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   676
        } else {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   677
            requestContentLen = 0;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   678
        }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   679
        OutgoingHeaders<Stream<T>> f = headerFrame(requestContentLen);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   680
        connection.sendFrame(f);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   681
        CompletableFuture<ExchangeImpl<T>> cf = new MinimalFuture<>();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   682
        cf.complete(this);  // #### good enough for now
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   683
        return cf;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   684
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   685
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   686
    @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   687
    void released() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   688
        if (streamid > 0) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   689
            if (debug.on()) debug.log("Released stream %d", streamid);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   690
            // remove this stream from the Http2Connection map.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   691
            connection.closeStream(streamid);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   692
        } else {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   693
            if (debug.on()) debug.log("Can't release stream %d", streamid);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   694
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   695
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   696
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   697
    @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   698
    void completed() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   699
        // There should be nothing to do here: the stream should have
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   700
        // been already closed (or will be closed shortly after).
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   701
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   702
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   703
    void registerStream(int id) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   704
        this.streamid = id;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   705
        connection.putStream(this, streamid);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   706
        if (debug.on()) debug.log("Registered stream %d", id);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   707
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   708
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   709
    void signalWindowUpdate() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   710
        RequestSubscriber subscriber = requestSubscriber;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   711
        assert subscriber != null;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   712
        if (debug.on()) debug.log("Signalling window update");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   713
        subscriber.sendScheduler.runOrSchedule();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   714
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   715
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   716
    static final ByteBuffer COMPLETED = ByteBuffer.allocate(0);
45708
3512073b446f 8178699: Fail to send async requests if server doesn't response the first one
xiaofeya
parents: 45531
diff changeset
   717
    class RequestSubscriber implements Flow.Subscriber<ByteBuffer> {
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   718
        // can be < 0 if the actual length is not known.
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   719
        private final long contentLength;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   720
        private volatile long remainingContentLength;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   721
        private volatile Subscription subscription;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   722
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   723
        // Holds the outgoing data. There will be at most 2 outgoing ByteBuffers.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   724
        //  1) The data that was published by the request body Publisher, and
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   725
        //  2) the COMPLETED sentinel, since onComplete can be invoked without demand.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   726
        final ConcurrentLinkedDeque<ByteBuffer> outgoing = new ConcurrentLinkedDeque<>();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   727
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   728
        private final AtomicReference<Throwable> errorRef = new AtomicReference<>();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   729
        // A scheduler used to honor window updates. Writing must be paused
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   730
        // when the window is exhausted, and resumed when the window acquires
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   731
        // some space. The sendScheduler makes it possible to implement this
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   732
        // behaviour in an asynchronous non-blocking way.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   733
        // See RequestSubscriber::trySend below.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   734
        final SequentialScheduler sendScheduler;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   735
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   736
        RequestSubscriber(long contentLen) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   737
            this.contentLength = contentLen;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   738
            this.remainingContentLength = contentLen;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   739
            this.sendScheduler =
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   740
                    SequentialScheduler.synchronizedScheduler(this::trySend);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   741
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   742
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   743
        @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   744
        public void onSubscribe(Flow.Subscription subscription) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   745
            if (this.subscription != null) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   746
                throw new IllegalStateException("already subscribed");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   747
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   748
            this.subscription = subscription;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   749
            if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   750
                debug.log("RequestSubscriber: onSubscribe, request 1");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   751
            subscription.request(1);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   752
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   753
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   754
        @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   755
        public void onNext(ByteBuffer item) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   756
            if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   757
                debug.log("RequestSubscriber: onNext(%d)", item.remaining());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   758
            int size = outgoing.size();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   759
            assert size == 0 : "non-zero size: " + size;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   760
            onNextImpl(item);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   761
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   762
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   763
        private void onNextImpl(ByteBuffer item) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   764
            // Got some more request body bytes to send.
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   765
            if (requestBodyCF.isDone()) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   766
                // stream already cancelled, probably in timeout
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   767
                sendScheduler.stop();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   768
                subscription.cancel();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   769
                return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   770
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   771
            outgoing.add(item);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   772
            sendScheduler.runOrSchedule();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   773
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   774
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   775
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   776
        public void onError(Throwable throwable) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   777
            if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   778
                debug.log(() -> "RequestSubscriber: onError: " + throwable);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   779
            // ensure that errors are handled within the flow.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   780
            if (errorRef.compareAndSet(null, throwable)) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   781
                sendScheduler.runOrSchedule();
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   782
            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   783
        }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   784
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   785
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   786
        public void onComplete() {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   787
            if (debug.on()) debug.log("RequestSubscriber: onComplete");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   788
            int size = outgoing.size();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   789
            assert size == 0 || size == 1 : "non-zero or one size: " + size;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   790
            // last byte of request body has been obtained.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   791
            // ensure that everything is completed within the flow.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   792
            onNextImpl(COMPLETED);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   793
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   794
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   795
        // Attempts to send the data, if any.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   796
        // Handles errors and completion state.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   797
        // Pause writing if the send window is exhausted, resume it if the
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   798
        // send window has some bytes that can be acquired.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   799
        void trySend() {
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   800
            try {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   801
                // handle errors raised by onError;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   802
                Throwable t = errorRef.get();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   803
                if (t != null) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   804
                    sendScheduler.stop();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   805
                    if (requestBodyCF.isDone()) return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   806
                    subscription.cancel();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   807
                    requestBodyCF.completeExceptionally(t);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   808
                    cancelImpl(t);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   809
                    return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   810
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   811
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   812
                do {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   813
                    // handle COMPLETED;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   814
                    ByteBuffer item = outgoing.peekFirst();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   815
                    if (item == null) return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   816
                    else if (item == COMPLETED) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   817
                        sendScheduler.stop();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   818
                        complete();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   819
                        return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   820
                    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   821
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   822
                    // handle bytes to send downstream
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   823
                    while (item.hasRemaining()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   824
                        if (debug.on()) debug.log("trySend: %d", item.remaining());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   825
                        assert !endStreamSent : "internal error, send data after END_STREAM flag";
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   826
                        DataFrame df = getDataFrame(item);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   827
                        if (df == null) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   828
                            if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   829
                                debug.log("trySend: can't send yet: %d", item.remaining());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   830
                            return; // the send window is exhausted: come back later
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   831
                        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   832
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   833
                        if (contentLength > 0) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   834
                            remainingContentLength -= df.getDataLength();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   835
                            if (remainingContentLength < 0) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   836
                                String msg = connection().getConnectionFlow()
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   837
                                        + " stream=" + streamid + " "
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   838
                                        + "[" + Thread.currentThread().getName() + "] "
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   839
                                        + "Too many bytes in request body. Expected: "
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   840
                                        + contentLength + ", got: "
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   841
                                        + (contentLength - remainingContentLength);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   842
                                connection.resetStream(streamid, ResetFrame.PROTOCOL_ERROR);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   843
                                throw new IOException(msg);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   844
                            } else if (remainingContentLength == 0) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   845
                                df.setFlag(DataFrame.END_STREAM);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   846
                                endStreamSent = true;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   847
                            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   848
                        }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   849
                        if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   850
                            debug.log("trySend: sending: %d", df.getDataLength());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   851
                        connection.sendDataFrame(df);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   852
                    }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   853
                    assert !item.hasRemaining();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   854
                    ByteBuffer b = outgoing.removeFirst();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   855
                    assert b == item;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   856
                } while (outgoing.peekFirst() != null);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   857
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   858
                if (debug.on()) debug.log("trySend: request 1");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   859
                subscription.request(1);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   860
            } catch (Throwable ex) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   861
                if (debug.on()) debug.log("trySend: ", ex);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   862
                sendScheduler.stop();
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   863
                subscription.cancel();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   864
                requestBodyCF.completeExceptionally(ex);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   865
                // need to cancel the stream to 1. tell the server
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   866
                // we don't want to receive any more data and
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   867
                // 2. ensure that the operation ref count will be
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   868
                // decremented on the HttpClient.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   869
                cancelImpl(ex);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   870
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   871
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   872
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   873
        private void complete() throws IOException {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   874
            long remaining = remainingContentLength;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   875
            long written = contentLength - remaining;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   876
            if (remaining > 0) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   877
                connection.resetStream(streamid, ResetFrame.PROTOCOL_ERROR);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   878
                // let trySend() handle the exception
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   879
                throw new IOException(connection().getConnectionFlow()
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   880
                                     + " stream=" + streamid + " "
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   881
                                     + "[" + Thread.currentThread().getName() +"] "
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   882
                                     + "Too few bytes returned by the publisher ("
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   883
                                              + written + "/"
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   884
                                              + contentLength + ")");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   885
            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   886
            if (!endStreamSent) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   887
                endStreamSent = true;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   888
                connection.sendDataFrame(getEmptyEndStreamDataFrame());
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   889
            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   890
            requestBodyCF.complete(null);
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
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   894
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   895
     * Send a RESET frame to tell server to stop sending data on this stream
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   896
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   897
    @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   898
    public CompletableFuture<Void> ignoreBody() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   899
        try {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   900
            connection.resetStream(streamid, ResetFrame.STREAM_CLOSED);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   901
            return MinimalFuture.completedFuture(null);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   902
        } catch (Throwable e) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   903
            Log.logTrace("Error resetting stream {0}", e.toString());
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   904
            return MinimalFuture.failedFuture(e);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   905
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   906
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   907
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   908
    DataFrame getDataFrame(ByteBuffer buffer) {
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   909
        int requestAmount = Math.min(connection.getMaxSendFrameSize(), buffer.remaining());
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   910
        // blocks waiting for stream send window, if exhausted
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   911
        int actualAmount = windowController.tryAcquire(requestAmount, streamid, this);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   912
        if (actualAmount <= 0) return null;
48703
3eae36c6baa5 8195823: Buffers given to response body subscribers should not contain unprocessed HTTP data
chegar
parents: 48523
diff changeset
   913
        ByteBuffer outBuf = Utils.sliceWithLimitedCapacity(buffer,  actualAmount);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   914
        DataFrame df = new DataFrame(streamid, 0 , outBuf);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   915
        return df;
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
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   918
    private DataFrame getEmptyEndStreamDataFrame()  {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   919
        return new DataFrame(streamid, DataFrame.END_STREAM, List.of());
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   920
    }
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
    /**
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   923
     * A List of responses relating to this stream. Normally there is only
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   924
     * one response, but intermediate responses like 100 are allowed
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   925
     * and must be passed up to higher level before continuing. Deals with races
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   926
     * such as if responses are returned before the CFs get created by
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   927
     * getResponseAsync()
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   928
     */
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   929
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   930
    final List<CompletableFuture<Response>> response_cfs = new ArrayList<>(5);
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
    @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   933
    CompletableFuture<Response> getResponseAsync(Executor executor) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   934
        CompletableFuture<Response> cf;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   935
        // The code below deals with race condition that can be caused when
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   936
        // completeResponse() is being called before getResponseAsync()
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   937
        synchronized (response_cfs) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   938
            if (!response_cfs.isEmpty()) {
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   939
                // This CompletableFuture was created by completeResponse().
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   940
                // it will be already completed.
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   941
                cf = response_cfs.remove(0);
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   942
                // if we find a cf here it should be already completed.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   943
                // finding a non completed cf should not happen. just assert it.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   944
                assert cf.isDone() : "Removing uncompleted response: could cause code to hang!";
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   945
            } else {
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   946
                // getResponseAsync() is called first. Create a CompletableFuture
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   947
                // that will be completed by completeResponse() when
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   948
                // completeResponse() is called.
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   949
                cf = new MinimalFuture<>();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   950
                response_cfs.add(cf);
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
        }
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   953
        if (executor != null && !cf.isDone()) {
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   954
            // protect from executing later chain of CompletableFuture operations from SelectorManager thread
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   955
            cf = cf.thenApplyAsync(r -> r, executor);
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
   956
        }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   957
        Log.logTrace("Response future (stream={0}) is: {1}", streamid, cf);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
   958
        PushGroup<?> pg = exchange.getPushGroup();
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   959
        if (pg != null) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   960
            // if an error occurs make sure it is recorded in the PushGroup
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   961
            cf = cf.whenComplete((t,e) -> pg.pushError(Utils.getCompletionCause(e)));
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   962
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   963
        return cf;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   964
    }
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
     * Completes the first uncompleted CF on list, and removes it. If there is no
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   968
     * uncompleted CF then creates one (completes it) and adds to list
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   969
     */
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   970
    void completeResponse(Response resp) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   971
        synchronized (response_cfs) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   972
            CompletableFuture<Response> cf;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   973
            int cfs_len = response_cfs.size();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   974
            for (int i=0; i<cfs_len; i++) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   975
                cf = response_cfs.get(i);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   976
                if (!cf.isDone()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   977
                    Log.logTrace("Completing response (streamid={0}): {1}",
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   978
                                 streamid, cf);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   979
                    cf.complete(resp);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   980
                    response_cfs.remove(cf);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   981
                    return;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
   982
                } // else we found the previous response: just leave it alone.
42460
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
            cf = MinimalFuture.completedFuture(resp);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   985
            Log.logTrace("Created completed future (streamid={0}): {1}",
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   986
                         streamid, cf);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   987
            response_cfs.add(cf);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   988
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   989
    }
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
    // methods to update state and remove stream when finished
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
    synchronized void requestSent() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   994
        requestSent = true;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   995
        if (responseReceived) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   996
            close();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   997
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
   998
    }
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
    synchronized void responseReceived() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1001
        responseReceived = true;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1002
        if (requestSent) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1003
            close();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1004
        }
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
    /**
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1008
     * same as above but for errors
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
    void completeResponseExceptionally(Throwable t) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1011
        synchronized (response_cfs) {
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
  1012
            // use index to avoid ConcurrentModificationException
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
  1013
            // caused by removing the CF from within the loop.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
  1014
            for (int i = 0; i < response_cfs.size(); i++) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
  1015
                CompletableFuture<Response> cf = response_cfs.get(i);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1016
                if (!cf.isDone()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1017
                    cf.completeExceptionally(t);
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents: 43999
diff changeset
  1018
                    response_cfs.remove(i);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1019
                    return;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1020
                }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1021
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1022
            response_cfs.add(MinimalFuture.failedFuture(t));
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1023
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1024
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1025
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1026
    CompletableFuture<Void> sendBodyImpl() {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1027
        requestBodyCF.whenComplete((v, t) -> requestSent());
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1028
        try {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1029
            if (requestPublisher != null) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1030
                final RequestSubscriber subscriber = new RequestSubscriber(requestContentLen);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1031
                requestPublisher.subscribe(requestSubscriber = subscriber);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1032
            } else {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1033
                // there is no request body, therefore the request is complete,
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1034
                // END_STREAM has already sent with outgoing headers
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1035
                requestBodyCF.complete(null);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1036
            }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1037
        } catch (Throwable t) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1038
            cancelImpl(t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1039
            requestBodyCF.completeExceptionally(t);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1040
        }
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1041
        return requestBodyCF;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1042
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1043
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1044
    @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1045
    void cancel() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1046
        cancel(new IOException("Stream " + streamid + " cancelled"));
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1047
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1048
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1049
    void onSubscriptionError(Throwable t) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1050
        errorRef.compareAndSet(null, t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1051
        if (debug.on()) debug.log("Got subscription error: %s", (Object)t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1052
        // This is the special case where the subscriber
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1053
        // has requested an illegal number of items.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1054
        // In this case, the error doesn't come from
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1055
        // upstream, but from downstream, and we need to
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1056
        // handle the error without waiting for the inputQ
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1057
        // to be exhausted.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1058
        stopRequested = true;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1059
        sched.runOrSchedule();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1060
    }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1061
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1062
    @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1063
    void cancel(IOException cause) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1064
        cancelImpl(cause);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1065
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1066
56531
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1067
    void connectionClosing(Throwable cause) {
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1068
        Flow.Subscriber<?> subscriber =
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1069
                responseSubscriber == null ? pendingResponseSubscriber : responseSubscriber;
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1070
        errorRef.compareAndSet(null, cause);
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1071
        if (subscriber != null && !sched.isStopped() && !inputQ.isEmpty()) {
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1072
            sched.runOrSchedule();
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1073
        } else cancelImpl(cause);
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1074
    }
15ff86a732ea http-client-branch: connection shutdown should no cancel streams immediately if data is pending in the stream queue.
dfuchs
parents: 56514
diff changeset
  1075
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1076
    // This method sends a RST_STREAM frame
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1077
    void cancelImpl(Throwable e) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1078
        errorRef.compareAndSet(null, e);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1079
        if (debug.on()) debug.log("cancelling stream {0}: {1}", streamid, e);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1080
        if (Log.trace()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1081
            Log.logTrace("cancelling stream {0}: {1}\n", streamid, e);
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
        boolean closing;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1084
        if (closing = !closed) { // assigning closing to !closed
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1085
            synchronized (this) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1086
                if (closing = !closed) { // assigning closing to !closed
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1087
                    closed=true;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1088
                }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1089
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1090
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1091
        if (closing) { // true if the stream has not been closed yet
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1092
            if (responseSubscriber != null || pendingResponseSubscriber != null)
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1093
                sched.runOrSchedule();
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1094
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1095
        completeResponseExceptionally(e);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1096
        if (!requestBodyCF.isDone()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1097
            requestBodyCF.completeExceptionally(errorRef.get()); // we may be sending the body..
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1098
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1099
        if (responseBodyCF != null) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1100
            responseBodyCF.completeExceptionally(errorRef.get());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1101
        }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1102
        try {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1103
            // will send a RST_STREAM frame
45531
fb3dbffad37b 8180044: java/net/httpclient/ManyRequests.java failed due to timeout
dfuchs
parents: 44639
diff changeset
  1104
            if (streamid != 0) {
56514
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
  1105
                e = Utils.getCompletionCause(e);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
  1106
                if (e instanceof EOFException) {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
  1107
                    // read EOF: no need to try & send reset
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
  1108
                    connection.closeStream(streamid);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
  1109
                } else {
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
  1110
                    connection.resetStream(streamid, ResetFrame.CANCEL);
6ef5ca8283a4 http-client-branch: RST_STREAM should be handled immediately if received before HEADERS, and subscriber.onError() should be called if received after that.
dfuchs
parents: 56507
diff changeset
  1111
                }
45531
fb3dbffad37b 8180044: java/net/httpclient/ManyRequests.java failed due to timeout
dfuchs
parents: 44639
diff changeset
  1112
            }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1113
        } catch (IOException ex) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1114
            Log.logError(ex);
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
    // This method doesn't send any frame
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1119
    void close() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1120
        if (closed) return;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1121
        synchronized(this) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1122
            if (closed) return;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1123
            closed = true;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1124
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1125
        Log.logTrace("Closing stream {0}", streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1126
        connection.closeStream(streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1127
        Log.logTrace("Stream {0} closed", streamid);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1128
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1129
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1130
    static class PushedStream<T> extends Stream<T> {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1131
        final PushGroup<T> pushGroup;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1132
        // push streams need the response CF allocated up front as it is
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1133
        // given directly to user via the multi handler callback function.
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1134
        final CompletableFuture<Response> pushCF;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1135
        CompletableFuture<HttpResponse<T>> responseCF;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1136
        final HttpRequestImpl pushReq;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1137
        HttpResponse.BodyHandler<T> pushHandler;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1138
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1139
        PushedStream(PushGroup<T> pushGroup,
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1140
                     Http2Connection connection,
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1141
                     Exchange<T> pushReq) {
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1142
            // ## no request body possible, null window controller
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1143
            super(connection, pushReq, null);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1144
            this.pushGroup = pushGroup;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1145
            this.pushReq = pushReq.request();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1146
            this.pushCF = new MinimalFuture<>();
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1147
            this.responseCF = new MinimalFuture<>();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1148
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1149
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1150
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1151
        CompletableFuture<HttpResponse<T>> responseCF() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1152
            return responseCF;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1153
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1154
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1155
        synchronized void setPushHandler(HttpResponse.BodyHandler<T> pushHandler) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1156
            this.pushHandler = pushHandler;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1157
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1158
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1159
        synchronized HttpResponse.BodyHandler<T> getPushHandler() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1160
            // ignored parameters to function can be used as BodyHandler
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1161
            return this.pushHandler;
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
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1164
        // Following methods call the super class but in case of
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1165
        // error record it in the PushGroup. The error method is called
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1166
        // with a null value when no error occurred (is a no-op)
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1167
        @Override
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1168
        CompletableFuture<ExchangeImpl<T>> sendBodyAsync() {
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1169
            return super.sendBodyAsync()
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1170
                        .whenComplete((ExchangeImpl<T> v, Throwable t)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1171
                                -> pushGroup.pushError(Utils.getCompletionCause(t)));
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1172
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1173
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1174
        @Override
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1175
        CompletableFuture<ExchangeImpl<T>> sendHeadersAsync() {
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1176
            return super.sendHeadersAsync()
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1177
                        .whenComplete((ExchangeImpl<T> ex, Throwable t)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1178
                                -> pushGroup.pushError(Utils.getCompletionCause(t)));
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1179
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1180
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1181
        @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1182
        CompletableFuture<Response> getResponseAsync(Executor executor) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1183
            CompletableFuture<Response> cf = pushCF.whenComplete(
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1184
                    (v, t) -> pushGroup.pushError(Utils.getCompletionCause(t)));
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1185
            if(executor!=null && !cf.isDone()) {
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1186
                cf  = cf.thenApplyAsync( r -> r, executor);
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1187
            }
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1188
            return cf;
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1189
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1190
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1191
        @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1192
        CompletableFuture<T> readBodyAsync(
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1193
                HttpResponse.BodyHandler<T> handler,
43999
4cc44dd9f14f 8164625: Pooled HttpConnection should be removed during close
prappo
parents: 43984
diff changeset
  1194
                boolean returnConnectionToPool,
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1195
                Executor executor)
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1196
        {
43999
4cc44dd9f14f 8164625: Pooled HttpConnection should be removed during close
prappo
parents: 43984
diff changeset
  1197
            return super.readBodyAsync(handler, returnConnectionToPool, executor)
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1198
                        .whenComplete((v, t) -> pushGroup.pushError(t));
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1199
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1200
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1201
        @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1202
        void completeResponse(Response r) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1203
            Log.logResponse(r::toString);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1204
            pushCF.complete(r); // not strictly required for push API
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1205
            // start reading the body using the obtained BodySubscriber
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1206
            CompletableFuture<Void> start = new MinimalFuture<>();
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1207
            start.thenCompose( v -> readBodyAsync(getPushHandler(), false, getExchange().executor()))
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1208
                .whenComplete((T body, Throwable t) -> {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1209
                    if (t != null) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1210
                        responseCF.completeExceptionally(t);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1211
                    } else {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1212
                        HttpResponseImpl<T> resp =
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1213
                                new HttpResponseImpl<>(r.request, r, null, body, getExchange());
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1214
                        responseCF.complete(resp);
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1215
                    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1216
                });
43984
f33383dcb1fb 8175274: Fix httpclient asynchronous usage
skuksenko
parents: 42460
diff changeset
  1217
            start.completeAsync(() -> null, getExchange().executor());
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1218
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1219
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1220
        @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1221
        void completeResponseExceptionally(Throwable t) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1222
            pushCF.completeExceptionally(t);
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
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1225
//        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1226
//        synchronized void responseReceived() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1227
//            super.responseReceived();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1228
//        }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1229
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1230
        // create and return the PushResponseImpl
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1231
        @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1232
        protected void handleResponse() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1233
            responseCode = (int)responseHeaders
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1234
                .firstValueAsLong(":status")
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1235
                .orElse(-1);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1236
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1237
            if (responseCode == -1) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1238
                completeResponseExceptionally(new IOException("No status code"));
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1239
            }
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
            this.response = new Response(
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1242
                pushReq, exchange, responseHeaders, connection(),
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1243
                responseCode, HttpClient.Version.HTTP_2);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1244
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1245
            /* TODO: review if needs to be removed
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1246
               the value is not used, but in case `content-length` doesn't parse
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1247
               as long, there will be NumberFormatException. If left as is, make
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1248
               sure code up the stack handles NFE correctly. */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1249
            responseHeaders.firstValueAsLong("content-length");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1250
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1251
            if (Log.headers()) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1252
                StringBuilder sb = new StringBuilder("RESPONSE HEADERS");
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1253
                sb.append(" (streamid=").append(streamid).append("):\n");
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1254
                Log.dumpHeaders(sb, "    ", responseHeaders);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1255
                Log.logHeaders(sb.toString());
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1256
            }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1257
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1258
            rspHeadersConsumer.reset();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1259
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1260
            // different implementations for normal streams and pushed streams
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1261
            completeResponse(response);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1262
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1263
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1264
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1265
    final class StreamWindowUpdateSender extends WindowUpdateSender {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1266
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1267
        StreamWindowUpdateSender(Http2Connection connection) {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1268
            super(connection);
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1269
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1270
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1271
        @Override
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1272
        int getStreamId() {
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1273
            return streamid;
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1274
        }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1275
    }
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1276
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1277
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1278
     * Returns true if this exchange was canceled.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1279
     * @return true if this exchange was canceled.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1280
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1281
    synchronized boolean isCanceled() {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1282
        return errorRef.get() != null;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1283
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1284
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1285
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1286
     * Returns the cause for which this exchange was canceled, if available.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1287
     * @return the cause for which this exchange was canceled, if available.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1288
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1289
    synchronized Throwable getCancelCause() {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1290
        return errorRef.get();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1291
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1292
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1293
    final String dbgString() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1294
        return connection.dbgString() + "/Stream("+streamid+")";
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
  1295
    }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1296
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1297
    private class HeadersConsumer extends Http2Connection.ValidatingHeadersConsumer {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1298
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1299
        void reset() {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1300
            super.reset();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1301
            responseHeaders.clear();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1302
        }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1303
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1304
        @Override
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1305
        public void onDecoded(CharSequence name, CharSequence value)
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1306
                throws UncheckedIOException
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1307
        {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1308
            String n = name.toString();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1309
            String v = value.toString();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1310
            super.onDecoded(n, v);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1311
            responseHeaders.addHeader(n, v);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1312
            if (Log.headers() && Log.trace()) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1313
                Log.logTrace("RECEIVED HEADER (streamid={0}): {1}: {2}",
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1314
                             streamid, n, v);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1315
            }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1316
        }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48703
diff changeset
  1317
    }
42460
7133f144981a 8170648: Move java.net.http package out of Java SE to incubator namespace
michaelm
parents:
diff changeset
  1318
}