test/jdk/java/net/httpclient/SmallTimeout.java
author dfuchs
Fri, 13 Apr 2018 10:49:17 +0100
branchhttp-client-branch
changeset 56423 ba64c30666cc
parent 56274 56e6b3f2e09f
child 56451 9585061fdb04
permissions -rw-r--r--
http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
     1
/*
56167
96fa4f49a9ff http-client-branch: CSR review commet - outboard pre-defined BP/BH/BS
chegar
parents: 56092
diff changeset
     2
 * Copyright (c) 2015, 2018, Oracle and/or its affiliates. All rights reserved.
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
     4
 *
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
     7
 * published by the Free Software Foundation.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
     8
 *
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    13
 * accompanied this code).
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    14
 *
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    18
 *
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    21
 * questions.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    22
 */
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    23
56265
ec34ae013fbe http-client-branch: tests should bind to the loopback only
chegar
parents: 56233
diff changeset
    24
import java.net.InetAddress;
56233
1753108d07b9 http-client-branch: setReuseAddress(false) for all test ServerSockets
chegar
parents: 56167
diff changeset
    25
import java.net.InetSocketAddress;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    26
import java.net.ServerSocket;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    27
import java.net.URI;
56089
42208b2f224e http-client-branch: move to standard package and module name
chegar
parents: 56082
diff changeset
    28
import java.net.http.HttpClient;
42208b2f224e http-client-branch: move to standard package and module name
chegar
parents: 56082
diff changeset
    29
import java.net.http.HttpRequest;
42208b2f224e http-client-branch: move to standard package and module name
chegar
parents: 56082
diff changeset
    30
import java.net.http.HttpResponse;
56167
96fa4f49a9ff http-client-branch: CSR review commet - outboard pre-defined BP/BH/BS
chegar
parents: 56092
diff changeset
    31
import java.net.http.HttpResponse.BodyHandlers;
56089
42208b2f224e http-client-branch: move to standard package and module name
chegar
parents: 56082
diff changeset
    32
import java.net.http.HttpTimeoutException;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    33
import java.time.Duration;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    34
import java.util.concurrent.CompletableFuture;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    35
import java.util.concurrent.ExecutorService;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    36
import java.util.concurrent.Executors;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    37
import java.util.concurrent.LinkedBlockingQueue;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    38
import static java.lang.System.out;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    39
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    40
/**
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    41
 * @test
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    42
 * @bug 8178147
56423
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
    43
 * @modules java.net.http/jdk.internal.net.http.common
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    44
 * @summary Ensures that small timeouts do not cause hangs due to race conditions
56274
56e6b3f2e09f http-client-branch: test for default impl of newWebSocketBuilder
chegar
parents: 56265
diff changeset
    45
 * @run main/othervm -Djdk.internal.httpclient.debug=true SmallTimeout
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    46
 */
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    47
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    48
// To enable logging use. Not enabled by default as it changes the dynamics
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    49
// of the test.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    50
// @run main/othervm -Djdk.httpclient.HttpClient.log=all,frames:all SmallTimeout
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    51
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    52
public class SmallTimeout {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    53
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    54
    static int[] TIMEOUTS = {2, 1, 3, 2, 100, 1};
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    55
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    56
    // A queue for placing timed out requests so that their order can be checked.
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    57
    static LinkedBlockingQueue<HttpResult> queue = new LinkedBlockingQueue<>();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    58
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    59
    static final class HttpResult {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    60
         final HttpRequest request;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    61
         final Throwable   failed;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    62
         HttpResult(HttpRequest request, Throwable   failed) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    63
             this.request = request;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    64
             this.failed = failed;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    65
         }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    66
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    67
         static HttpResult of(HttpRequest request) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    68
             return new HttpResult(request, null);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    69
         }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    70
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    71
         static HttpResult of(HttpRequest request, Throwable t) {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    72
             return new HttpResult(request, t);
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    73
         }
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    74
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
    75
    }
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    76
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    77
    static volatile boolean error;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    78
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    79
    public static void main(String[] args) throws Exception {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    80
        HttpClient client = HttpClient.newHttpClient();
56423
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
    81
        ReferenceTracker.INSTANCE.track(client);
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    82
56423
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
    83
        Throwable failed = null;
56233
1753108d07b9 http-client-branch: setReuseAddress(false) for all test ServerSockets
chegar
parents: 56167
diff changeset
    84
        try (ServerSocket ss = new ServerSocket()) {
1753108d07b9 http-client-branch: setReuseAddress(false) for all test ServerSockets
chegar
parents: 56167
diff changeset
    85
            ss.setReuseAddress(false);
56265
ec34ae013fbe http-client-branch: tests should bind to the loopback only
chegar
parents: 56233
diff changeset
    86
            ss.bind(new InetSocketAddress(InetAddress.getLoopbackAddress(), 0));
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    87
            int port = ss.getLocalPort();
56265
ec34ae013fbe http-client-branch: tests should bind to the loopback only
chegar
parents: 56233
diff changeset
    88
            URI uri = new URI("http://localhost:" + port + "/");
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    89
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    90
            HttpRequest[] requests = new HttpRequest[TIMEOUTS.length];
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    91
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    92
            out.println("--- TESTING Async");
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    93
            for (int i = 0; i < TIMEOUTS.length; i++) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    94
                requests[i] = HttpRequest.newBuilder(uri)
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    95
                                         .timeout(Duration.ofMillis(TIMEOUTS[i]))
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    96
                                         .GET()
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    97
                                         .build();
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    98
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
    99
                final HttpRequest req = requests[i];
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   100
                CompletableFuture<HttpResponse<Object>> response = client
56167
96fa4f49a9ff http-client-branch: CSR review commet - outboard pre-defined BP/BH/BS
chegar
parents: 56092
diff changeset
   101
                    .sendAsync(req, BodyHandlers.replacing(null))
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   102
                    .whenComplete((HttpResponse<Object> r, Throwable t) -> {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   103
                        Throwable cause = null;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   104
                        if (r != null) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   105
                            out.println("Unexpected response: " + r);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   106
                            cause = new RuntimeException("Unexpected response");
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   107
                            error = true;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   108
                        }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   109
                        if (t != null) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   110
                            if (!(t.getCause() instanceof HttpTimeoutException)) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   111
                                out.println("Wrong exception type:" + t.toString());
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   112
                                Throwable c = t.getCause() == null ? t : t.getCause();
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   113
                                c.printStackTrace();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   114
                                cause = c;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   115
                                error = true;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   116
                            } else {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   117
                                out.println("Caught expected timeout: " + t.getCause());
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   118
                            }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   119
                        }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   120
                        if (t == null && r == null) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   121
                            out.println("Both response and throwable are null!");
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   122
                            cause = new RuntimeException("Both response and throwable are null!");
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   123
                            error = true;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   124
                        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   125
                        queue.add(HttpResult.of(req,cause));
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   126
                    });
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   127
            }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   128
            System.out.println("All requests submitted. Waiting ...");
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   129
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   130
            checkReturn(requests);
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   131
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   132
            if (error)
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   133
                throw new RuntimeException("Failed. Check output");
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   134
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   135
            // Repeat blocking in separate threads. Use queue to wait.
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   136
            out.println("--- TESTING Sync");
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   137
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   138
            // For running blocking response tasks
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   139
            ExecutorService executor = Executors.newCachedThreadPool();
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   140
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   141
            for (int i = 0; i < TIMEOUTS.length; i++) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   142
                requests[i] = HttpRequest.newBuilder(uri)
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   143
                                         .timeout(Duration.ofMillis(TIMEOUTS[i]))
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   144
                                         .GET()
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   145
                                         .build();
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   146
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   147
                final HttpRequest req = requests[i];
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   148
                executor.execute(() -> {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   149
                    Throwable cause = null;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   150
                    try {
56167
96fa4f49a9ff http-client-branch: CSR review commet - outboard pre-defined BP/BH/BS
chegar
parents: 56092
diff changeset
   151
                        client.send(req, BodyHandlers.replacing(null));
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   152
                    } catch (HttpTimeoutException e) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   153
                        out.println("Caught expected timeout: " + e);
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   154
                    } catch (Throwable ee) {
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   155
                        Throwable c = ee.getCause() == null ? ee : ee.getCause();
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   156
                        c.printStackTrace();
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   157
                        cause = c;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   158
                        error = true;
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   159
                    } finally {
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   160
                        queue.offer(HttpResult.of(req, cause));
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   161
                    }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   162
                });
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   163
            }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   164
            System.out.println("All requests submitted. Waiting ...");
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   165
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   166
            checkReturn(requests);
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   167
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   168
            executor.shutdownNow();
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   169
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   170
            if (error)
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   171
                throw new RuntimeException("Failed. Check output");
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   172
56423
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   173
        } catch (Throwable t) {
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   174
            failed = t;
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   175
            throw t;
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   176
        } finally {
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   177
            try {
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   178
                Thread.sleep(100);
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   179
            } catch (InterruptedException t) {
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   180
                // ignore;
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   181
            }
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   182
            AssertionError trackFailed = ReferenceTracker.INSTANCE.check(500);
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   183
            if (trackFailed != null) {
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   184
                if (failed != null) {
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   185
                    failed.addSuppressed(trackFailed);
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   186
                    if (failed instanceof Exception) throw (Exception) failed;
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   187
                    if (failed instanceof Error) throw (Exception) failed;
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   188
                }
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   189
                throw trackFailed;
ba64c30666cc http-client-branch: add test coverage and better diagnosis for HttpClientImpl operations ref count
dfuchs
parents: 56274
diff changeset
   190
            }
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   191
        }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   192
    }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   193
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   194
    static void checkReturn(HttpRequest[] requests) throws InterruptedException {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   195
        // wait for exceptions and check order
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   196
        boolean ok = true;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   197
        for (int j = 0; j < TIMEOUTS.length; j++) {
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   198
            HttpResult res = queue.take();
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   199
            HttpRequest req = res.request;
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   200
            out.println("Got request from queue " + req + ", order: " + getRequest(req, requests)
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   201
                         + (res.failed == null ? "" : " failed: " + res.failed));
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   202
            ok = ok && res.failed == null;
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   203
        }
48083
b1c1b4ef4be2 8191494: Refresh incubating HTTP Client
chegar
parents: 47216
diff changeset
   204
        out.println("Return " + (ok ? "ok" : "nok"));
44639
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   205
    }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   206
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   207
    /** Returns the index of the request in the array. */
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   208
    static String getRequest(HttpRequest req, HttpRequest[] requests) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   209
        for (int i=0; i<requests.length; i++) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   210
            if (req == requests[i]) {
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   211
                return "r" + i;
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   212
            }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   213
        }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   214
        throw new AssertionError("Unknown request: " + req);
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   215
    }
5c2838d882a5 8178147: Race conditions in timeout handling code in http/2 incubator client
dfuchs
parents:
diff changeset
   216
}