src/java.net.http/share/classes/jdk/internal/net/http/common/SSLFlowDelegate.java
author dfuchs
Wed, 16 Oct 2019 14:50:53 +0100
changeset 58649 6b6bf0de534b
parent 53350 a47b8125b7cc
permissions -rw-r--r--
8231449: HttpClient’s client ssl certificate authentication seems to be broken. Summary: SSLFlowDelegate.Reader and SubscriberWrapper are changed to better cooperate on when more demand should be requested from upstream. The issue encountered in this scenario was triggered by a large certificate which caused the SSLFlowDelegate to stop requesting data from upstream during the handshake although the engine handshake status was NEED_UNWRAP. Reviewed-by: chegar
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
     1
/*
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
     2
 * Copyright (c) 2017, 2019, Oracle and/or its affiliates. All rights reserved.
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
     4
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    10
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    15
 * accompanied this code).
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    16
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    20
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    23
 * questions.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    24
 */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    25
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    26
package jdk.internal.net.http.common;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    27
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    28
import jdk.internal.net.http.common.SubscriberWrapper.SchedulingAction;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    29
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    30
import javax.net.ssl.SSLEngine;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    31
import javax.net.ssl.SSLEngineResult;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    32
import javax.net.ssl.SSLEngineResult.HandshakeStatus;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    33
import javax.net.ssl.SSLEngineResult.Status;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    34
import javax.net.ssl.SSLException;
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
    35
import javax.net.ssl.SSLHandshakeException;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    36
import java.io.IOException;
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
    37
import java.lang.ref.Reference;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
    38
import java.lang.ref.ReferenceQueue;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
    39
import java.lang.ref.WeakReference;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    40
import java.nio.ByteBuffer;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    41
import java.util.ArrayList;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    42
import java.util.Collections;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    43
import java.util.Iterator;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    44
import java.util.LinkedList;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    45
import java.util.List;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    46
import java.util.concurrent.CompletableFuture;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    47
import java.util.concurrent.ConcurrentLinkedQueue;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    48
import java.util.concurrent.Executor;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    49
import java.util.concurrent.Flow;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    50
import java.util.concurrent.Flow.Subscriber;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    51
import java.util.concurrent.atomic.AtomicInteger;
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
    52
import java.util.function.Consumer;
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
    53
import java.util.function.IntBinaryOperator;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    54
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    55
/**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    56
 * Implements SSL using two SubscriberWrappers.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    57
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    58
 * <p> Constructor takes two Flow.Subscribers: one that receives the network
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    59
 * data (after it has been encrypted by SSLFlowDelegate) data, and one that
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    60
 * receives the application data (before it has been encrypted by SSLFlowDelegate).
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    61
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    62
 * <p> Methods upstreamReader() and upstreamWriter() return the corresponding
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    63
 * Flow.Subscribers containing Flows for the encrypted/decrypted upstream data.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    64
 * See diagram below.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    65
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    66
 * <p> How Flow.Subscribers are used in this class, and where they come from:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    67
 * <pre>
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    68
 * {@code
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    69
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    70
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    71
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    72
 * --------->  data flow direction
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    73
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    74
 *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    75
 *                         +------------------+
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    76
 *        upstreamWriter   |                  | downWriter
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    77
 *        ---------------> |                  | ------------>
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    78
 *  obtained from this     |                  | supplied to constructor
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    79
 *                         | SSLFlowDelegate  |
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    80
 *        downReader       |                  | upstreamReader
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    81
 *        <--------------- |                  | <--------------
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    82
 * supplied to constructor |                  | obtained from this
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    83
 *                         +------------------+
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    84
 *
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    85
 * Errors are reported to the downReader Flow.Subscriber
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    86
 *
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    87
 * }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    88
 * </pre>
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    89
 */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    90
public class SSLFlowDelegate {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    91
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    92
    final Logger debug =
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
    93
            Utils.getDebugLogger(this::dbgString, Utils.DEBUG);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
    94
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
    95
    private static final ByteBuffer SENTINEL = Utils.EMPTY_BYTEBUFFER;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
    96
    private static final ByteBuffer HS_TRIGGER = ByteBuffer.allocate(0);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
    97
    // When handshake is in progress trying to wrap may produce no bytes.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
    98
    private static final ByteBuffer NOTHING = ByteBuffer.allocate(0);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
    99
    private static final String monProp = Utils.getProperty("jdk.internal.httpclient.monitorFlowDelegate");
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   100
    private static final boolean isMonitored =
52902
e3398b2e1ab0 8214971: Replace use of string.equals("") with isEmpty()
rriggs
parents: 50985
diff changeset
   101
            monProp != null && (monProp.isEmpty() || monProp.equalsIgnoreCase("true"));
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   102
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   103
    final Executor exec;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   104
    final Reader reader;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   105
    final Writer writer;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   106
    final SSLEngine engine;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   107
    final String tubeName; // hack
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   108
    final CompletableFuture<String> alpnCF; // completes on initial handshake
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   109
    final Monitorable monitor = isMonitored ? this::monitor : null; // prevent GC until SSLFD is stopped
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   110
    volatile boolean close_notify_received;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   111
    final CompletableFuture<Void> readerCF;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   112
    final CompletableFuture<Void> writerCF;
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   113
    final CompletableFuture<Void> stopCF;
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   114
    final Consumer<ByteBuffer> recycler;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   115
    static AtomicInteger scount = new AtomicInteger(1);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   116
    final int id;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   117
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   118
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   119
     * Creates an SSLFlowDelegate fed from two Flow.Subscribers. Each
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   120
     * Flow.Subscriber requires an associated {@link CompletableFuture}
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   121
     * for errors that need to be signaled from downstream to upstream.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   122
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   123
    public SSLFlowDelegate(SSLEngine engine,
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   124
                           Executor exec,
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   125
                           Subscriber<? super List<ByteBuffer>> downReader,
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   126
                           Subscriber<? super List<ByteBuffer>> downWriter)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   127
    {
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   128
        this(engine, exec, null, downReader, downWriter);
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   129
    }
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   130
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   131
    /**
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   132
     * Creates an SSLFlowDelegate fed from two Flow.Subscribers. Each
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   133
     * Flow.Subscriber requires an associated {@link CompletableFuture}
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   134
     * for errors that need to be signaled from downstream to upstream.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   135
     */
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   136
    public SSLFlowDelegate(SSLEngine engine,
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   137
            Executor exec,
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   138
            Consumer<ByteBuffer> recycler,
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   139
            Subscriber<? super List<ByteBuffer>> downReader,
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   140
            Subscriber<? super List<ByteBuffer>> downWriter)
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   141
        {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   142
        this.id = scount.getAndIncrement();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   143
        this.tubeName = String.valueOf(downWriter);
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   144
        this.recycler = recycler;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   145
        this.reader = new Reader();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   146
        this.writer = new Writer();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   147
        this.engine = engine;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   148
        this.exec = exec;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   149
        this.handshakeState = new AtomicInteger(NOT_HANDSHAKING);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   150
        this.readerCF = reader.completion();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   151
        this.writerCF = reader.completion();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   152
        readerCF.exceptionally(this::stopOnError);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   153
        writerCF.exceptionally(this::stopOnError);
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   154
        this.stopCF = CompletableFuture.allOf(reader.completion(), writer.completion())
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   155
            .thenRun(this::normalStop);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   156
        this.alpnCF = new MinimalFuture<>();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   157
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   158
        // connect the Reader to the downReader and the
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   159
        // Writer to the downWriter.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   160
        connect(downReader, downWriter);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   161
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   162
        if (isMonitored) Monitor.add(monitor);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   163
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   164
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   165
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   166
     * Returns true if the SSLFlowDelegate has detected a TLS
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   167
     * close_notify from the server.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   168
     * @return true, if a close_notify was detected.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   169
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   170
    public boolean closeNotifyReceived() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   171
        return close_notify_received;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   172
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   173
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   174
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   175
     * Connects the read sink (downReader) to the SSLFlowDelegate Reader,
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   176
     * and the write sink (downWriter) to the SSLFlowDelegate Writer.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   177
     * Called from within the constructor. Overwritten by SSLTube.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   178
     *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   179
     * @param downReader  The left hand side read sink (typically, the
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   180
     *                    HttpConnection read subscriber).
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   181
     * @param downWriter  The right hand side write sink (typically
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   182
     *                    the SocketTube write subscriber).
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   183
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   184
    void connect(Subscriber<? super List<ByteBuffer>> downReader,
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   185
                 Subscriber<? super List<ByteBuffer>> downWriter) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   186
        this.reader.subscribe(downReader);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   187
        this.writer.subscribe(downWriter);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   188
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   189
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   190
   /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   191
    * Returns a CompletableFuture<String> which completes after
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   192
    * the initial handshake completes, and which contains the negotiated
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   193
    * alpn.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   194
    */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   195
    public CompletableFuture<String> alpn() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   196
        return alpnCF;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   197
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   198
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   199
    private void setALPN() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   200
        // Handshake is finished. So, can retrieve the ALPN now
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   201
        if (alpnCF.isDone())
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   202
            return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   203
        String alpn = engine.getApplicationProtocol();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   204
        if (debug.on()) debug.log("setALPN = %s", alpn);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   205
        alpnCF.complete(alpn);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   206
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   207
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   208
    public String monitor() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   209
        StringBuilder sb = new StringBuilder();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   210
        sb.append("SSL: id ").append(id);
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   211
        sb.append(" ").append(dbgString());
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   212
        sb.append(" HS state: " + states(handshakeState));
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   213
        sb.append(" Engine state: " + engine.getHandshakeStatus().toString());
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   214
        if (stateList != null) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   215
            sb.append(" LL : ");
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   216
            for (String s : stateList) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   217
                sb.append(s).append(" ");
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   218
            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   219
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   220
        sb.append("\r\n");
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   221
        sb.append("Reader:: ").append(reader.toString());
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   222
        sb.append("\r\n");
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   223
        sb.append("Writer:: ").append(writer.toString());
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   224
        sb.append("\r\n===================================");
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   225
        return sb.toString();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   226
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   227
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   228
    protected SchedulingAction enterReadScheduling() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   229
        return SchedulingAction.CONTINUE;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   230
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   231
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   232
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   233
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   234
     * Processing function for incoming data. Pass it thru SSLEngine.unwrap().
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   235
     * Any decrypted buffers returned to be passed downstream.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   236
     * Status codes:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   237
     *     NEED_UNWRAP: do nothing. Following incoming data will contain
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   238
     *                  any required handshake data
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   239
     *     NEED_WRAP: call writer.addData() with empty buffer
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   240
     *     NEED_TASK: delegate task to executor
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   241
     *     BUFFER_OVERFLOW: allocate larger output buffer. Repeat unwrap
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   242
     *     BUFFER_UNDERFLOW: keep buffer and wait for more data
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   243
     *     OK: return generated buffers.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   244
     *
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   245
     * Upstream subscription strategy is to try and keep no more than
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   246
     * TARGET_BUFSIZE bytes in readBuf
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   247
     */
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   248
    final class Reader extends SubscriberWrapper implements FlowTube.TubeSubscriber {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   249
        // Maximum record size is 16k.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   250
        // Because SocketTube can feeds us up to 3 16K buffers,
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   251
        // then setting this size to 16K means that the readBuf
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   252
        // can store up to 64K-1 (16K-1 + 3*16K)
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   253
        static final int TARGET_BUFSIZE = 16 * 1024;
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   254
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   255
        final SequentialScheduler scheduler;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   256
        volatile ByteBuffer readBuf;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   257
        volatile boolean completing;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   258
        final Object readBufferLock = new Object();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   259
        final Logger debugr = Utils.getDebugLogger(this::dbgString, Utils.DEBUG);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   260
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   261
        private final class ReaderDownstreamPusher implements Runnable {
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   262
            @Override
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   263
            public void run() {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   264
                processData();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   265
            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   266
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   267
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   268
        Reader() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   269
            super();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   270
            scheduler = SequentialScheduler.synchronizedScheduler(
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   271
                    new ReaderDownstreamPusher());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   272
            this.readBuf = ByteBuffer.allocate(1024);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   273
            readBuf.limit(0); // keep in read mode
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   274
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   275
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   276
        @Override
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   277
        public boolean supportsRecycling() {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   278
            return recycler != null;
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   279
        }
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   280
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   281
        protected SchedulingAction enterScheduling() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   282
            return enterReadScheduling();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   283
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   284
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   285
        public final String dbgString() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   286
            return "SSL Reader(" + tubeName + ")";
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   287
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   288
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   289
        /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   290
         * entry point for buffers delivered from upstream Subscriber
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   291
         */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   292
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   293
        public void incoming(List<ByteBuffer> buffers, boolean complete) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   294
            if (debugr.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   295
                debugr.log("Adding %d bytes to read buffer",
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   296
                        Utils.remaining(buffers));
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   297
            addToReadBuf(buffers, complete);
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   298
            scheduler.runOrSchedule(exec);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   299
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   300
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   301
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   302
        public String toString() {
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   303
            return "READER: " + super.toString() + ", readBuf: " + readBuf.toString()
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   304
                    + ", count: " + count.toString() + ", scheduler: "
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   305
                    + (scheduler.isStopped() ? "stopped" : "running")
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   306
                    + ", status: " + lastUnwrapStatus
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   307
                    + ", handshakeState: " + handshakeState.get()
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   308
                    + ", engine: " + engine.getHandshakeStatus();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   309
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   310
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   311
        private void reallocReadBuf() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   312
            int sz = readBuf.capacity();
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   313
            ByteBuffer newb = ByteBuffer.allocate(sz * 2);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   314
            readBuf.flip();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   315
            Utils.copy(readBuf, newb);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   316
            readBuf = newb;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   317
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   318
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   319
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   320
        protected long upstreamWindowUpdate(long currentWindow, long downstreamQsize) {
58649
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   321
            if (needsMoreData()) {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   322
                // run the scheduler to see if more data should be requested
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   323
                if (debugr.on()) {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   324
                    int remaining = readBuf.remaining();
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   325
                    if (remaining > TARGET_BUFSIZE) {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   326
                        // just some logging to check how much we have in the read buffer
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   327
                        debugr.log("readBuf has more than TARGET_BUFSIZE: %d",
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   328
                                remaining);
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   329
                    }
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   330
                }
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   331
                scheduler.runOrSchedule();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   332
            }
58649
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   333
            return 0; // we will request more from the scheduler loop (processData).
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   334
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   335
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   336
        // readBuf is kept ready for reading outside of this method
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   337
        private void addToReadBuf(List<ByteBuffer> buffers, boolean complete) {
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   338
            assert Utils.remaining(buffers) > 0 || buffers.isEmpty();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   339
            synchronized (readBufferLock) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   340
                for (ByteBuffer buf : buffers) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   341
                    readBuf.compact();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   342
                    while (readBuf.remaining() < buf.remaining())
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   343
                        reallocReadBuf();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   344
                    readBuf.put(buf);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   345
                    readBuf.flip();
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   346
                    // should be safe to call inside lock
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   347
                    // since the only implementation
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   348
                    // offers the buffer to an unbounded queue.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   349
                    // WARNING: do not touch buf after this point!
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   350
                    if (recycler != null) recycler.accept(buf);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   351
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   352
                if (complete) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   353
                    this.completing = complete;
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   354
                    minBytesRequired = 0;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   355
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   356
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   357
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   358
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   359
        void schedule() {
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   360
            scheduler.runOrSchedule(exec);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   361
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   362
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   363
        void stop() {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   364
            if (debugr.on()) debugr.log("stop");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   365
            scheduler.stop();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   366
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   367
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   368
        AtomicInteger count = new AtomicInteger(0);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   369
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   370
        // minimum number of bytes required to call unwrap.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   371
        // Usually this is 0, unless there was a buffer underflow.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   372
        // In this case we need to wait for more bytes than what
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   373
        // we had before calling unwrap() again.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   374
        volatile int minBytesRequired;
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   375
58649
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   376
        // We might need to request more data if:
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   377
        //  - we have a subscription from upstream
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   378
        //  - and we don't have enough data to decrypt in the read buffer
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   379
        //  - *and* - either we're handshaking, and more data is required (NEED_UNWRAP),
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   380
        //          - or we have demand from downstream, but we have nothing decrypted
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   381
        //            to forward downstream.
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   382
        boolean needsMoreData() {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   383
            if (upstreamSubscription != null && readBuf.remaining() <= minBytesRequired &&
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   384
                    (engine.getHandshakeStatus() == HandshakeStatus.NEED_UNWRAP
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   385
                            || !downstreamSubscription.demand.isFulfilled() && hasNoOutputData())) {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   386
                return true;
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   387
            }
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   388
            return false;
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   389
        }
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   390
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   391
        // If the readBuf has not enough data, and we either need to
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   392
        // unwrap (handshaking) or we have demand from downstream,
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   393
        // then request more data
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   394
        void requestMoreDataIfNeeded() {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   395
            if (needsMoreData()) {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   396
                // request more will only request more if our
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   397
                // demand from upstream is fulfilled
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   398
                requestMore();
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   399
            }
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   400
        }
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   401
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   402
        // work function where it all happens
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   403
        final void processData() {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   404
            try {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   405
                if (debugr.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   406
                    debugr.log("processData:"
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   407
                            + " readBuf remaining:" + readBuf.remaining()
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   408
                            + ", state:" + states(handshakeState)
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   409
                            + ", engine handshake status:" + engine.getHandshakeStatus());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   410
                int len;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   411
                boolean complete = false;
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   412
                while (readBuf.remaining() > (len = minBytesRequired)) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   413
                    boolean handshaking = false;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   414
                    try {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   415
                        EngineResult result;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   416
                        synchronized (readBufferLock) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   417
                            complete = this.completing;
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   418
                            if (debugr.on()) debugr.log("Unwrapping: %s", readBuf.remaining());
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   419
                            // Unless there is a BUFFER_UNDERFLOW, we should try to
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   420
                            // unwrap any number of bytes. Set minBytesRequired to 0:
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   421
                            // we only need to do that if minBytesRequired is not already 0.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   422
                            len = len > 0 ? minBytesRequired = 0 : len;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   423
                            result = unwrapBuffer(readBuf);
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   424
                            len = readBuf.remaining();
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   425
                            if (debugr.on()) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   426
                                debugr.log("Unwrapped: result: %s", result.result);
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   427
                                debugr.log("Unwrapped: consumed: %s", result.bytesConsumed());
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   428
                            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   429
                        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   430
                        if (result.bytesProduced() > 0) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   431
                            if (debugr.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   432
                                debugr.log("sending %d", result.bytesProduced());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   433
                            count.addAndGet(result.bytesProduced());
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   434
                            outgoing(result.destBuffer, false);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   435
                        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   436
                        if (result.status() == Status.BUFFER_UNDERFLOW) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   437
                            if (debugr.on()) debugr.log("BUFFER_UNDERFLOW");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   438
                            // not enough data in the read buffer...
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   439
                            // no need to try to unwrap again unless we get more bytes
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   440
                            // than minBytesRequired = len in the read buffer.
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   441
                            synchronized (readBufferLock) {
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   442
                                minBytesRequired = len;
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   443
                                // more bytes could already have been added...
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   444
                                assert readBuf.remaining() >= len;
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   445
                                // check if we have received some data, and if so
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   446
                                // we can just re-spin the loop
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   447
                                if (readBuf.remaining() > len) continue;
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   448
                                else if (this.completing) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   449
                                    if (debug.on()) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   450
                                        debugr.log("BUFFER_UNDERFLOW with EOF," +
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   451
                                                " %d bytes non decrypted.", len);
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   452
                                    }
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   453
                                    // The channel won't send us any more data, and
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   454
                                    // we are in underflow: we need to fail.
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   455
                                    throw new IOException("BUFFER_UNDERFLOW with EOF, "
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   456
                                            + len + " bytes non decrypted.");
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   457
                                }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   458
                            }
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   459
                            // request more data and return.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   460
                            requestMore();
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   461
                            return;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   462
                        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   463
                        if (complete && result.status() == Status.CLOSED) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   464
                            if (debugr.on()) debugr.log("Closed: completing");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   465
                            outgoing(Utils.EMPTY_BB_LIST, true);
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   466
                            // complete ALPN if not yet completed
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   467
                            setALPN();
58649
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   468
                            requestMoreDataIfNeeded();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   469
                            return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   470
                        }
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   471
                        if (result.handshaking()) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   472
                            handshaking = true;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   473
                            if (debugr.on()) debugr.log("handshaking");
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   474
                            if (doHandshake(result, READER)) continue; // need unwrap
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   475
                            else break; // doHandshake will have triggered the write scheduler if necessary
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   476
                        } else {
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   477
                            if (trySetALPN()) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   478
                                resumeActivity();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   479
                            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   480
                        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   481
                    } catch (IOException ex) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   482
                        errorCommon(ex);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   483
                        handleError(ex);
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   484
                        return;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   485
                    }
58649
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   486
                    if (handshaking && !complete) {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   487
                        requestMoreDataIfNeeded();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   488
                        return;
58649
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   489
                    }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   490
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   491
                if (!complete) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   492
                    synchronized (readBufferLock) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   493
                        complete = this.completing && !readBuf.hasRemaining();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   494
                    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   495
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   496
                if (complete) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   497
                    if (debugr.on()) debugr.log("completing");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   498
                    // Complete the alpnCF, if not already complete, regardless of
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   499
                    // whether or not the ALPN is available, there will be no more
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   500
                    // activity.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   501
                    setALPN();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   502
                    outgoing(Utils.EMPTY_BB_LIST, true);
58649
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   503
                } else {
6b6bf0de534b 8231449: HttpClient’s client ssl certificate authentication seems to be broken.
dfuchs
parents: 53350
diff changeset
   504
                    requestMoreDataIfNeeded();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   505
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   506
            } catch (Throwable ex) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   507
                errorCommon(ex);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   508
                handleError(ex);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   509
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   510
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   511
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   512
        private volatile Status lastUnwrapStatus;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   513
        EngineResult unwrapBuffer(ByteBuffer src) throws IOException {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   514
            ByteBuffer dst = getAppBuffer();
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   515
            int len = src.remaining();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   516
            while (true) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   517
                SSLEngineResult sslResult = engine.unwrap(src, dst);
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   518
                switch (lastUnwrapStatus = sslResult.getStatus()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   519
                    case BUFFER_OVERFLOW:
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   520
                        // may happen if app size buffer was changed, or if
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   521
                        // our 'adaptiveBufferSize' guess was too small for
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   522
                        // the current payload. In that case, update the
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   523
                        // value of applicationBufferSize, and allocate a
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   524
                        // buffer of that size, which we are sure will be
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   525
                        // big enough to decode whatever needs to be
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   526
                        // decoded. We will later update adaptiveBufferSize
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   527
                        // in OK: below.
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   528
                        int appSize = applicationBufferSize =
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   529
                                engine.getSession().getApplicationBufferSize();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   530
                        ByteBuffer b = ByteBuffer.allocate(appSize + dst.position());
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   531
                        dst.flip();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   532
                        b.put(dst);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   533
                        dst = b;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   534
                        break;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   535
                    case CLOSED:
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   536
                        assert dst.position() == 0;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   537
                        return doClosure(new EngineResult(sslResult));
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   538
                    case BUFFER_UNDERFLOW:
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   539
                        // handled implicitly by compaction/reallocation of readBuf
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   540
                        assert dst.position() == 0;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   541
                        return new EngineResult(sslResult);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   542
                    case OK:
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   543
                        int size = dst.position();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   544
                        if (debug.on()) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   545
                            debugr.log("Decoded " + size + " bytes out of " + len
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   546
                                    + " into buffer of " + dst.capacity()
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   547
                                    + " remaining to decode: " + src.remaining());
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   548
                        }
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   549
                        // if the record payload was bigger than what was originally
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   550
                        // allocated, then sets the adaptiveAppBufferSize to size
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   551
                        // and we will use that new size as a guess for the next app
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   552
                        // buffer.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   553
                        if (size > adaptiveAppBufferSize) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   554
                            adaptiveAppBufferSize = ((size + 7) >>> 3) << 3;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   555
                        }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   556
                        dst.flip();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   557
                        return new EngineResult(sslResult, dst);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   558
                }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   559
            }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   560
        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   561
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   562
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   563
    public interface Monitorable {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   564
        public String getInfo();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   565
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   566
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   567
    public static class Monitor extends Thread {
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   568
        final List<WeakReference<Monitorable>> list;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   569
        final List<FinalMonitorable> finalList;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   570
        final ReferenceQueue<Monitorable> queue = new ReferenceQueue<>();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   571
        static Monitor themon;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   572
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   573
        static {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   574
            themon = new Monitor();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   575
            themon.start(); // uncomment to enable Monitor
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   576
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   577
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   578
        // An instance used to temporarily store the
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   579
        // last observable state of a monitorable object.
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   580
        // When Monitor.remove(o) is called, we replace
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   581
        // 'o' with a FinalMonitorable whose reference
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   582
        // will be enqueued after the last observable state
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   583
        // has been printed.
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   584
        final class FinalMonitorable implements Monitorable {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   585
            final String finalState;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   586
            FinalMonitorable(Monitorable o) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   587
                finalState = o.getInfo();
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   588
                finalList.add(this);
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   589
            }
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   590
            @Override
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   591
            public String getInfo() {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   592
                finalList.remove(this);
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   593
                return finalState;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   594
            }
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   595
        }
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   596
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   597
        Monitor() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   598
            super("Monitor");
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   599
            setDaemon(true);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   600
            list = Collections.synchronizedList(new LinkedList<>());
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   601
            finalList = new ArrayList<>(); // access is synchronized on list above
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   602
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   603
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   604
        void addTarget(Monitorable o) {
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   605
            list.add(new WeakReference<>(o, queue));
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   606
        }
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   607
        void removeTarget(Monitorable o) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   608
            // It can take a long time for GC to clean up references.
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   609
            // Calling Monitor.remove() early helps removing noise from the
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   610
            // logs/
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   611
            synchronized (list) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   612
                Iterator<WeakReference<Monitorable>> it = list.iterator();
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   613
                while (it.hasNext()) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   614
                    Monitorable m = it.next().get();
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   615
                    if (m == null) it.remove();
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   616
                    if (o == m) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   617
                        it.remove();
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   618
                        break;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   619
                    }
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   620
                }
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   621
                FinalMonitorable m = new FinalMonitorable(o);
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   622
                addTarget(m);
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   623
                Reference.reachabilityFence(m);
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   624
            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   625
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   626
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   627
        public static void add(Monitorable o) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   628
            themon.addTarget(o);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   629
        }
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   630
        public static void remove(Monitorable o) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   631
            themon.removeTarget(o);
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   632
        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   633
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   634
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   635
        public void run() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   636
            System.out.println("Monitor starting");
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   637
            try {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   638
                while (true) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   639
                    Thread.sleep(20 * 1000);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   640
                    synchronized (list) {
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   641
                        Reference<? extends Monitorable> expired;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   642
                        while ((expired = queue.poll()) != null) list.remove(expired);
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   643
                        for (WeakReference<Monitorable> ref : list) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   644
                            Monitorable o = ref.get();
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   645
                            if (o == null) continue;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   646
                            if (o instanceof FinalMonitorable) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   647
                                ref.enqueue();
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   648
                            }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   649
                            System.out.println(o.getInfo());
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   650
                            System.out.println("-------------------------");
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   651
                        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   652
                    }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   653
                    System.out.println("--o-o-o-o-o-o-o-o-o-o-o-o-o-o-");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   654
                }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   655
            } catch (InterruptedException e) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   656
                System.out.println("Monitor exiting with " + e);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   657
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   658
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   659
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   660
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   661
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   662
     * Processing function for outgoing data. Pass it thru SSLEngine.wrap()
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   663
     * Any encrypted buffers generated are passed downstream to be written.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   664
     * Status codes:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   665
     *     NEED_UNWRAP: call reader.addData() with empty buffer
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   666
     *     NEED_WRAP: call addData() with empty buffer
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   667
     *     NEED_TASK: delegate task to executor
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   668
     *     BUFFER_OVERFLOW: allocate larger output buffer. Repeat wrap
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   669
     *     BUFFER_UNDERFLOW: shouldn't happen on writing side
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   670
     *     OK: return generated buffers
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   671
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   672
    class Writer extends SubscriberWrapper {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   673
        final SequentialScheduler scheduler;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   674
        // queues of buffers received from upstream waiting
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   675
        // to be processed by the SSLEngine
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   676
        final List<ByteBuffer> writeList;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   677
        final Logger debugw =  Utils.getDebugLogger(this::dbgString, Utils.DEBUG);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   678
        volatile boolean completing;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   679
        boolean completed; // only accessed in processData
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   680
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   681
        class WriterDownstreamPusher extends SequentialScheduler.CompleteRestartableTask {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   682
            @Override public void run() { processData(); }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   683
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   684
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   685
        Writer() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   686
            super();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   687
            writeList = Collections.synchronizedList(new LinkedList<>());
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   688
            scheduler = new SequentialScheduler(new WriterDownstreamPusher());
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   689
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   690
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   691
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   692
        protected void incoming(List<ByteBuffer> buffers, boolean complete) {
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   693
            assert complete ? buffers == Utils.EMPTY_BB_LIST : true;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   694
            assert buffers != Utils.EMPTY_BB_LIST ? complete == false : true;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   695
            if (complete) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   696
                if (debugw.on()) debugw.log("adding SENTINEL");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   697
                completing = true;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   698
                writeList.add(SENTINEL);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   699
            } else {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   700
                writeList.addAll(buffers);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   701
            }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   702
            if (debugw.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   703
                debugw.log("added " + buffers.size()
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   704
                           + " (" + Utils.remaining(buffers)
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   705
                           + " bytes) to the writeList");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   706
            scheduler.runOrSchedule();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   707
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   708
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   709
        public final String dbgString() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   710
            return "SSL Writer(" + tubeName + ")";
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   711
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   712
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   713
        protected void onSubscribe() {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   714
            if (debugw.on()) debugw.log("onSubscribe initiating handshaking");
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   715
            addData(HS_TRIGGER);  // initiates handshaking
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   716
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   717
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   718
        void schedule() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   719
            scheduler.runOrSchedule();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   720
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   721
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   722
        void stop() {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   723
            if (debugw.on()) debugw.log("stop");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   724
            scheduler.stop();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   725
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   726
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   727
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   728
        public boolean closing() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   729
            return closeNotifyReceived();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   730
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   731
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   732
        private boolean isCompleting() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   733
            return completing;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   734
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   735
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   736
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   737
        protected long upstreamWindowUpdate(long currentWindow, long downstreamQsize) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   738
            if (writeList.size() > 10)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   739
                return 0;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   740
            else
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   741
                return super.upstreamWindowUpdate(currentWindow, downstreamQsize);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   742
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   743
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   744
        private boolean hsTriggered() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   745
            synchronized(writeList) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   746
                for (ByteBuffer b : writeList)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   747
                    if (b == HS_TRIGGER)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   748
                        return true;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   749
                return false;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   750
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   751
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   752
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   753
        void triggerWrite() {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   754
            synchronized (writeList) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   755
                if (writeList.isEmpty()) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   756
                    writeList.add(HS_TRIGGER);
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   757
                }
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   758
            }
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   759
            scheduler.runOrSchedule();
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   760
        }
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   761
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   762
        private void processData() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   763
            boolean completing = isCompleting();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   764
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   765
            try {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   766
                if (debugw.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   767
                    debugw.log("processData, writeList remaining:"
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   768
                                + Utils.remaining(writeList) + ", hsTriggered:"
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   769
                                + hsTriggered() + ", needWrap:" + needWrap());
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   770
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   771
                while (Utils.remaining(writeList) > 0 || hsTriggered() || needWrap()) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   772
                    ByteBuffer[] outbufs = writeList.toArray(Utils.EMPTY_BB_ARRAY);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   773
                    EngineResult result = wrapBuffers(outbufs);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   774
                    if (debugw.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   775
                        debugw.log("wrapBuffer returned %s", result.result);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   776
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   777
                    if (result.status() == Status.CLOSED) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   778
                        if (!upstreamCompleted) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   779
                            upstreamCompleted = true;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   780
                            upstreamSubscription.cancel();
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   781
                            // complete ALPN if not yet completed
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   782
                            setALPN();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   783
                        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   784
                        if (result.bytesProduced() <= 0)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   785
                            return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   786
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   787
                        if (!completing && !completed) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   788
                            completing = this.completing = true;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   789
                            // There could still be some outgoing data in outbufs.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   790
                            writeList.add(SENTINEL);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   791
                        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   792
                    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   793
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   794
                    boolean handshaking = false;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   795
                    if (result.handshaking()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   796
                        if (debugw.on()) debugw.log("handshaking");
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   797
                        doHandshake(result, WRITER);  // ok to ignore return
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   798
                        handshaking = true;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   799
                    } else {
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   800
                        if (trySetALPN()) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   801
                            resumeActivity();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   802
                        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   803
                    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   804
                    cleanList(writeList); // tidy up the source list
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   805
                    sendResultBytes(result);
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   806
                    if (handshaking) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   807
                        if (!completing && needWrap()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   808
                            continue;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   809
                        } else {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   810
                            return;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   811
                        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   812
                    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   813
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   814
                if (completing && Utils.remaining(writeList) == 0) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   815
                    if (!completed) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   816
                        completed = true;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   817
                        writeList.clear();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   818
                        outgoing(Utils.EMPTY_BB_LIST, true);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   819
                    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   820
                    return;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   821
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   822
                if (writeList.isEmpty() && needWrap()) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   823
                    writer.addData(HS_TRIGGER);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   824
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   825
            } catch (Throwable ex) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   826
                errorCommon(ex);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   827
                handleError(ex);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   828
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   829
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   830
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   831
        // The SSLEngine insists on being given a buffer that is at least
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   832
        // SSLSession.getPacketBufferSize() long (usually 16K). If given
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   833
        // a smaller buffer it will go in BUFFER_OVERFLOW, even if it only
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   834
        // has 6 bytes to wrap. Typical usage shows that for GET we
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   835
        // usually produce an average of ~ 100 bytes.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   836
        // To avoid wasting space, and because allocating and zeroing
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   837
        // 16K buffers for encoding 6 bytes is costly, we are reusing the
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   838
        // same writeBuffer to interact with SSLEngine.wrap().
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   839
        // If the SSLEngine produces less than writeBuffer.capacity() / 2,
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   840
        // then we copy off the bytes to a smaller buffer that we send
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   841
        // downstream. Otherwise, we send the writeBuffer downstream
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   842
        // and will allocate a new one next time.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   843
        volatile ByteBuffer writeBuffer;
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   844
        private volatile Status lastWrappedStatus;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   845
        @SuppressWarnings("fallthrough")
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   846
        EngineResult wrapBuffers(ByteBuffer[] src) throws SSLException {
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   847
            long len = Utils.remaining(src);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   848
            if (debugw.on())
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   849
                debugw.log("wrapping " + len + " bytes");
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   850
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   851
            ByteBuffer dst = writeBuffer;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   852
            if (dst == null) dst = writeBuffer = getNetBuffer();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   853
            assert dst.position() == 0 : "buffer position is " + dst.position();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   854
            assert dst.hasRemaining() : "buffer has no remaining space: capacity=" + dst.capacity();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   855
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   856
            while (true) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   857
                SSLEngineResult sslResult = engine.wrap(src, dst);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   858
                if (debugw.on()) debugw.log("SSLResult: " + sslResult);
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   859
                switch (lastWrappedStatus = sslResult.getStatus()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   860
                    case BUFFER_OVERFLOW:
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   861
                        // Shouldn't happen. We allocated buffer with packet size
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   862
                        // get it again if net buffer size was changed
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   863
                        if (debugw.on()) debugw.log("BUFFER_OVERFLOW");
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   864
                        int netSize = packetBufferSize
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
   865
                                = engine.getSession().getPacketBufferSize();
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   866
                        ByteBuffer b = writeBuffer = ByteBuffer.allocate(netSize + dst.position());
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   867
                        dst.flip();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   868
                        b.put(dst);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   869
                        dst = b;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   870
                        break; // try again
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   871
                    case CLOSED:
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   872
                        if (debugw.on()) debugw.log("CLOSED");
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   873
                        // fallthrough. There could be some remaining data in dst.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   874
                        // CLOSED will be handled by the caller.
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   875
                    case OK:
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   876
                        final ByteBuffer dest;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   877
                        if (dst.position() == 0) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   878
                            dest = NOTHING; // can happen if handshake is in progress
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   879
                        } else if (dst.position() < dst.capacity() / 2) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   880
                            // less than half the buffer was used.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   881
                            // copy off the bytes to a smaller buffer, and keep
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   882
                            // the writeBuffer for next time.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   883
                            dst.flip();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   884
                            dest = Utils.copyAligned(dst);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   885
                            dst.clear();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   886
                        } else {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   887
                            // more than half the buffer was used.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   888
                            // just send that buffer downstream, and we will
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   889
                            // get a new writeBuffer next time it is needed.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   890
                            dst.flip();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   891
                            dest = dst;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   892
                            writeBuffer = null;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   893
                        }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   894
                        if (debugw.on())
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   895
                            debugw.log("OK => produced: %d bytes into %d, not wrapped: %d",
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   896
                                       dest.remaining(),  dest.capacity(), Utils.remaining(src));
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   897
                        return new EngineResult(sslResult, dest);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   898
                    case BUFFER_UNDERFLOW:
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   899
                        // Shouldn't happen.  Doesn't returns when wrap()
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   900
                        // underflow handled externally
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   901
                        // assert false : "Buffer Underflow";
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   902
                        if (debug.on()) debug.log("BUFFER_UNDERFLOW");
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   903
                        return new EngineResult(sslResult);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   904
                    default:
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   905
                        if (debugw.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   906
                            debugw.log("result: %s", sslResult.getStatus());
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   907
                        assert false : "result:" + sslResult.getStatus();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   908
                }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   909
            }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   910
        }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   911
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   912
        private boolean needWrap() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   913
            return engine.getHandshakeStatus() == HandshakeStatus.NEED_WRAP;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   914
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   915
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   916
        private void sendResultBytes(EngineResult result) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   917
            if (result.bytesProduced() > 0) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   918
                if (debugw.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   919
                    debugw.log("Sending %d bytes downstream",
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   920
                               result.bytesProduced());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   921
                outgoing(result.destBuffer, false);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   922
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   923
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   924
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   925
        @Override
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   926
        public String toString() {
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   927
            return "WRITER: " + super.toString()
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   928
                    + ", writeList size: " + Integer.toString(writeList.size())
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   929
                    + ", scheduler: " + (scheduler.isStopped() ? "stopped" : "running")
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   930
                    + ", status: " + lastWrappedStatus;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   931
                    //" writeList: " + writeList.toString();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   932
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   933
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   934
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   935
    private void handleError(Throwable t) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   936
        if (debug.on()) debug.log("handleError", t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   937
        readerCF.completeExceptionally(t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   938
        writerCF.completeExceptionally(t);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   939
        // no-op if already completed
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   940
        alpnCF.completeExceptionally(t);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   941
        reader.stop();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   942
        writer.stop();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   943
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   944
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   945
    boolean stopped;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   946
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   947
    private synchronized void normalStop() {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   948
        if (stopped)
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   949
            return;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   950
        stopped = true;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   951
        reader.stop();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   952
        writer.stop();
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   953
        // make sure the alpnCF is completed.
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   954
        if (!alpnCF.isDone()) {
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   955
            Throwable alpn = new SSLHandshakeException(
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   956
                    "Connection closed before successful ALPN negotiation");
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   957
            alpnCF.completeExceptionally(alpn);
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   958
        }
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
   959
        if (isMonitored) Monitor.remove(monitor);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   960
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   961
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   962
    private Void stopOnError(Throwable error) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   963
        // maybe log, etc
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   964
        // ensure the ALPN is completed
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   965
        // We could also do this in SSLTube.SSLSubscriberWrapper
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   966
        // onError/onComplete - with the caveat that the ALP CF
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   967
        // would get completed externally. Doing it here keeps
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   968
        // it all inside SSLFlowDelegate.
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   969
        if (!alpnCF.isDone()) {
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   970
            alpnCF.completeExceptionally(error);
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
   971
        }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   972
        normalStop();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   973
        return null;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   974
    }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   975
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   976
    private void cleanList(List<ByteBuffer> l) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   977
        synchronized (l) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   978
            Iterator<ByteBuffer> iter = l.iterator();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   979
            while (iter.hasNext()) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   980
                ByteBuffer b = iter.next();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   981
                if (!b.hasRemaining() && b != SENTINEL) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   982
                    iter.remove();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   983
                }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   984
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   985
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   986
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   987
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   988
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   989
     * States for handshake. We avoid races when accessing/updating the AtomicInt
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   990
     * because updates always schedule an additional call to both the read()
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   991
     * and write() functions.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   992
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   993
    private static final int NOT_HANDSHAKING = 0;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
   994
    private static final int HANDSHAKING = 1;
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
   995
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   996
    // Bit flags
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   997
    // a thread is currently executing tasks
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   998
    private static final int DOING_TASKS = 4;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
   999
    // a thread wants to execute tasks, while another thread is executing
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1000
    private static final int REQUESTING_TASKS = 8;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1001
    private static final int TASK_BITS = 12; // Both bits
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1002
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1003
    private static final int READER = 1;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1004
    private static final int WRITER = 2;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1005
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1006
    private static String states(AtomicInteger state) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1007
        int s = state.get();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1008
        StringBuilder sb = new StringBuilder();
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1009
        int x = s & ~TASK_BITS;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1010
        switch (x) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1011
            case NOT_HANDSHAKING:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1012
                sb.append(" NOT_HANDSHAKING ");
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1013
                break;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1014
            case HANDSHAKING:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1015
                sb.append(" HANDSHAKING ");
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1016
                break;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1017
            default:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1018
                throw new InternalError();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1019
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1020
        if ((s & DOING_TASKS) > 0)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1021
            sb.append("|DOING_TASKS");
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1022
        if ((s & REQUESTING_TASKS) > 0)
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1023
            sb.append("|REQUESTING_TASKS");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1024
        return sb.toString();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1025
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1026
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1027
    private void resumeActivity() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1028
        reader.schedule();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1029
        writer.schedule();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1030
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1031
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1032
    final AtomicInteger handshakeState;
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1033
    final ConcurrentLinkedQueue<String> stateList =
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1034
            debug.on() ? new ConcurrentLinkedQueue<>() : null;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1035
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1036
    // Atomically executed to update task bits. Sets either DOING_TASKS or REQUESTING_TASKS
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1037
    // depending on previous value
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1038
    private static final IntBinaryOperator REQUEST_OR_DO_TASKS = (current, ignored) -> {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1039
        if ((current & DOING_TASKS) == 0)
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1040
            return DOING_TASKS | (current & HANDSHAKING);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1041
        else
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1042
            return DOING_TASKS | REQUESTING_TASKS | (current & HANDSHAKING);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1043
    };
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1044
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1045
    // Atomically executed to update task bits. Sets DOING_TASKS if REQUESTING was set
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1046
    // clears bits if not.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1047
    private static final IntBinaryOperator FINISH_OR_DO_TASKS = (current, ignored) -> {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1048
        if ((current & REQUESTING_TASKS) != 0)
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1049
            return DOING_TASKS | (current & HANDSHAKING);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1050
        // clear both bits
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1051
        return (current & HANDSHAKING);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1052
    };
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1053
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1054
    private boolean doHandshake(EngineResult r, int caller) {
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1055
        // unconditionally sets the HANDSHAKING bit, while preserving task bits
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1056
        handshakeState.getAndAccumulate(0, (current, unused) -> HANDSHAKING | (current & TASK_BITS));
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1057
        if (stateList != null && debug.on()) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1058
            stateList.add(r.handshakeStatus().toString());
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1059
            stateList.add(Integer.toString(caller));
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1060
        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1061
        switch (r.handshakeStatus()) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1062
            case NEED_TASK:
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1063
                int s = handshakeState.accumulateAndGet(0, REQUEST_OR_DO_TASKS);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1064
                if ((s & REQUESTING_TASKS) > 0) { // someone else is or will do tasks
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1065
                    return false;
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1066
                }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1067
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1068
                if (debug.on()) debug.log("obtaining and initiating task execution");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1069
                List<Runnable> tasks = obtainTasks();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1070
                executeTasks(tasks);
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1071
                return false;  // executeTasks will resume activity
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1072
            case NEED_WRAP:
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1073
                if (caller == READER) {
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1074
                    writer.triggerWrite();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1075
                    return false;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1076
                }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1077
                break;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1078
            case NEED_UNWRAP:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1079
            case NEED_UNWRAP_AGAIN:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1080
                // do nothing else
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1081
                // receiving-side data will trigger unwrap
50985
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
  1082
                if (caller == WRITER) {
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
  1083
                    reader.schedule();
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
  1084
                    return false;
cd41f34e548c 8206001: Enable TLS1.3 by default in Http Client
michaelm
parents: 50681
diff changeset
  1085
                }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1086
                break;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1087
            default:
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1088
                throw new InternalError("Unexpected handshake status:"
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1089
                                        + r.handshakeStatus());
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1090
        }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1091
        return true;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1092
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1093
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1094
    private List<Runnable> obtainTasks() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1095
        List<Runnable> l = new ArrayList<>();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1096
        Runnable r;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1097
        while ((r = engine.getDelegatedTask()) != null) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1098
            l.add(r);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1099
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1100
        return l;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1101
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1102
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1103
    private void executeTasks(List<Runnable> tasks) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1104
        exec.execute(() -> {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1105
            try {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1106
                List<Runnable> nextTasks = tasks;
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1107
                if (debug.on()) debug.log("#tasks to execute: " + Integer.toString(nextTasks.size()));
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1108
                do {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1109
                    nextTasks.forEach(Runnable::run);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1110
                    if (engine.getHandshakeStatus() == HandshakeStatus.NEED_TASK) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1111
                        nextTasks = obtainTasks();
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1112
                    } else {
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1113
                        int s = handshakeState.accumulateAndGet(0, FINISH_OR_DO_TASKS);
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1114
                        if ((s & DOING_TASKS) != 0) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1115
                            if (debug.on()) debug.log("re-running tasks (B)");
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1116
                            nextTasks = obtainTasks();
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1117
                            continue;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1118
                        }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1119
                        break;
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1120
                    }
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1121
                } while (true);
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1122
                if (debug.on()) debug.log("finished task execution");
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1123
                HandshakeStatus hs = engine.getHandshakeStatus();
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1124
                if (hs == HandshakeStatus.FINISHED || hs == HandshakeStatus.NOT_HANDSHAKING) {
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1125
                    // We're no longer handshaking, try setting ALPN
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1126
                    trySetALPN();
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1127
                }
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1128
                resumeActivity();
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1129
            } catch (Throwable t) {
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1130
                handleError(t);
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1131
            }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1132
        });
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1133
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1134
53350
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1135
    boolean trySetALPN() {
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1136
        // complete ALPN CF if needed.
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1137
        if ((handshakeState.getAndSet(NOT_HANDSHAKING) & ~DOING_TASKS) == HANDSHAKING) {
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1138
            applicationBufferSize = engine.getSession().getApplicationBufferSize();
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1139
            packetBufferSize = engine.getSession().getPacketBufferSize();
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1140
            setALPN();
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1141
            return true;
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1142
        }
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1143
        return false;
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1144
    }
a47b8125b7cc 8217094: HttpClient SSL race if a socket IOException is raised before ALPN is available
dfuchs
parents: 52902
diff changeset
  1145
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1146
    // FIXME: acknowledge a received CLOSE request from peer
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1147
    EngineResult doClosure(EngineResult r) throws IOException {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1148
        if (debug.on())
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1149
            debug.log("doClosure(%s): %s [isOutboundDone: %s, isInboundDone: %s]",
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1150
                      r.result, engine.getHandshakeStatus(),
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1151
                      engine.isOutboundDone(), engine.isInboundDone());
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1152
        if (engine.getHandshakeStatus() == HandshakeStatus.NEED_WRAP) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1153
            // we have received TLS close_notify and need to send
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1154
            // an acknowledgement back. We're calling doHandshake
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1155
            // to finish the close handshake.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1156
            if (engine.isInboundDone() && !engine.isOutboundDone()) {
49765
ee6f7a61f3a5 8197564: HTTP Client implementation
chegar
parents: 48083
diff changeset
  1157
                if (debug.on()) debug.log("doClosure: close_notify received");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1158
                close_notify_received = true;
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1159
                if (!writer.scheduler.isStopped()) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1160
                    doHandshake(r, READER);
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1161
                } else {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1162
                    // We have received closed notify, but we
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1163
                    // won't be able to send the acknowledgement.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1164
                    // Nothing more will come from the socket either,
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1165
                    // so mark the reader as completed.
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1166
                    synchronized (reader.readBufferLock) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1167
                        reader.completing = true;
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1168
                    }
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1169
                }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1170
            }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1171
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1172
        return r;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1173
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1174
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1175
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1176
     * Returns the upstream Flow.Subscriber of the reading (incoming) side.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1177
     * This flow must be given the encrypted data read from upstream (eg socket)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1178
     * before it is decrypted.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1179
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1180
    public Flow.Subscriber<List<ByteBuffer>> upstreamReader() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1181
        return reader;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1182
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1183
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1184
    /**
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1185
     * Returns the upstream Flow.Subscriber of the writing (outgoing) side.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1186
     * This flow contains the plaintext data before it is encrypted.
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1187
     */
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1188
    public Flow.Subscriber<List<ByteBuffer>> upstreamWriter() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1189
        return writer;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1190
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1191
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1192
    public boolean resumeReader() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1193
        return reader.signalScheduling();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1194
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1195
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1196
    public void resetReaderDemand() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1197
        reader.resetDownstreamDemand();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1198
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1199
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1200
    static class EngineResult {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1201
        final SSLEngineResult result;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1202
        final ByteBuffer destBuffer;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1203
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1204
        // normal result
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1205
        EngineResult(SSLEngineResult result) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1206
            this(result, null);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1207
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1208
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1209
        EngineResult(SSLEngineResult result, ByteBuffer destBuffer) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1210
            this.result = result;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1211
            this.destBuffer = destBuffer;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1212
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1213
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1214
        boolean handshaking() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1215
            HandshakeStatus s = result.getHandshakeStatus();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1216
            return s != HandshakeStatus.FINISHED
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1217
                   && s != HandshakeStatus.NOT_HANDSHAKING
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1218
                   && result.getStatus() != Status.CLOSED;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1219
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1220
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1221
        boolean needUnwrap() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1222
            HandshakeStatus s = result.getHandshakeStatus();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1223
            return s == HandshakeStatus.NEED_UNWRAP;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1224
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1225
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1226
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1227
        int bytesConsumed() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1228
            return result.bytesConsumed();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1229
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1230
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1231
        int bytesProduced() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1232
            return result.bytesProduced();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1233
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1234
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1235
        SSLEngineResult.HandshakeStatus handshakeStatus() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1236
            return result.getHandshakeStatus();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1237
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1238
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1239
        SSLEngineResult.Status status() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1240
            return result.getStatus();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1241
        }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1242
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1243
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1244
    // The maximum network buffer size negotiated during
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1245
    // the handshake. Usually 16K.
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1246
    volatile int packetBufferSize;
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1247
    final ByteBuffer getNetBuffer() {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1248
        int netSize = packetBufferSize;
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1249
        if (netSize <= 0) {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1250
            packetBufferSize = netSize = engine.getSession().getPacketBufferSize();
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1251
        }
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1252
        return ByteBuffer.allocate(netSize);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1253
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1254
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1255
    // The maximum application buffer size negotiated during
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1256
    // the handshake. Usually close to 16K.
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1257
    volatile int applicationBufferSize;
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1258
    // Despite of the maximum applicationBufferSize negotiated
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1259
    // above, TLS records usually have a much smaller payload.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1260
    // The adaptativeAppBufferSize records the max payload
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1261
    // ever decoded, and we use that as a guess for how big
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1262
    // a buffer we will need for the next payload.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1263
    // This avoids allocating and zeroing a 16K buffer for
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1264
    // nothing...
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1265
    volatile int adaptiveAppBufferSize;
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1266
    final ByteBuffer getAppBuffer() {
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1267
        int appSize = applicationBufferSize;
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1268
        if (appSize <= 0) {
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1269
            applicationBufferSize = appSize
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1270
                    = engine.getSession().getApplicationBufferSize();
49944
4690a2871b44 8202423: Small HTTP Client refresh
chegar
parents: 49765
diff changeset
  1271
        }
50681
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1272
        int size = adaptiveAppBufferSize;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1273
        if (size <= 0) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1274
            size = 512; // start with 512 this is usually enough for handshaking / headers
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1275
        } else if (size > appSize) {
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1276
            size = appSize;
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1277
        }
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1278
        // will cause a BUFFER_OVERFLOW if not big enough, but
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1279
        // that's OK.
4254bed3c09d 8204679: HTTP Client refresh
chegar
parents: 49944
diff changeset
  1280
        return ByteBuffer.allocate(size);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1281
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1282
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1283
    final String dbgString() {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1284
        return "SSLFlowDelegate(" + tubeName + ")";
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1285
    }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents:
diff changeset
  1286
}