aboutsummaryrefslogtreecommitdiffstats
path: root/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/proxy/ConfigServerRestExecutorImplTest.java
blob: 313485677ef57b53bfe191200e3fd86908ed6ced (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
// Copyright Vespa.ai. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
package com.yahoo.vespa.hosted.controller.proxy;

import ai.vespa.http.HttpURL.Path;
import ai.vespa.util.http.hc4.SslConnectionSocketFactory;
import com.github.tomakehurst.wiremock.junit5.WireMockExtension;
import com.github.tomakehurst.wiremock.stubbing.Scenario;
import com.yahoo.container.jdisc.HttpRequest;
import com.yahoo.container.jdisc.HttpResponse;
import com.yahoo.yolean.concurrent.Sleeper;
import org.apache.http.protocol.HttpContext;
import org.apache.http.protocol.HttpCoreContext;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.extension.RegisterExtension;

import java.io.ByteArrayOutputStream;
import java.net.URI;
import java.util.HashMap;
import java.util.Map;
import java.util.Set;

import static com.github.tomakehurst.wiremock.client.WireMock.aResponse;
import static com.github.tomakehurst.wiremock.client.WireMock.get;
import static com.github.tomakehurst.wiremock.client.WireMock.getRequestedFor;
import static com.github.tomakehurst.wiremock.client.WireMock.urlEqualTo;
import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.options;
import static org.junit.jupiter.api.Assertions.assertEquals;

/**
 * @author mpolden
 */
public class ConfigServerRestExecutorImplTest {

    @RegisterExtension
    public final WireMockExtension wireMock = WireMockExtension.newInstance().options(options().dynamicPort()).failOnUnmatchedRequests(true).build();

    @Test
    void proxy_with_retries() throws Exception {
        var connectionReuseStrategy = new CountingConnectionReuseStrategy(Set.of("127.0.0.1"));
        var proxy = new ConfigServerRestExecutorImpl(SslConnectionSocketFactory.of(), Sleeper.NOOP, connectionReuseStrategy);

        URI url = url();
        String path = url.getPath();
        stubRequests(path);

        HttpRequest request = HttpRequest.createTestRequest(url.toString(), com.yahoo.jdisc.http.HttpRequest.Method.GET);
        ProxyRequest proxyRequest = ProxyRequest.tryOne(url, Path.parse(path), request);

        // Request is retried
        HttpResponse response = proxy.handle(proxyRequest);
        wireMock.verify(3, getRequestedFor(urlEqualTo(path)));
        assertEquals(200, response.getStatus());
        ByteArrayOutputStream out = new ByteArrayOutputStream();
        response.render(out);
        assertEquals("OK", out.toString());

        // No connections are reused as host is a VIP
        assertEquals(0, connectionReuseStrategy.reusedConnections.get(url.getHost()).intValue());
    }

    @Test
    void proxy_without_connection_reuse() throws Exception {
        var connectionReuseStrategy = new CountingConnectionReuseStrategy(Set.of());
        var proxy = new ConfigServerRestExecutorImpl(SslConnectionSocketFactory.of(), Sleeper.NOOP, connectionReuseStrategy);
        URI url = url();
        String path = url.getPath();
        stubRequests(path);

        HttpRequest request = HttpRequest.createTestRequest(url.toString(), com.yahoo.jdisc.http.HttpRequest.Method.GET);
        ProxyRequest proxyRequest = ProxyRequest.tryOne(url, Path.parse(path), request);

        // Connections are reused
        assertEquals(200, proxy.handle(proxyRequest).getStatus());
        assertEquals(3, connectionReuseStrategy.reusedConnections.get(url.getHost()).intValue());
    }

    private URI url() {
        return URI.create("http://127.0.0.1:" + wireMock.getPort() + "/");
    }

    private void stubRequests(String path) {
        String retryScenario = "Retry scenario";
        String retryRequest = "Retry request 1";
        String retryRequestAgain = "Retry request 2";

        wireMock.stubFor(get(urlEqualTo(path)).inScenario(retryScenario)
                                              .whenScenarioStateIs(Scenario.STARTED)
                                              .willSetStateTo(retryRequest)
                                              .willReturn(aResponse().withStatus(500)));

        wireMock.stubFor(get(urlEqualTo(path)).inScenario(retryScenario)
                                              .whenScenarioStateIs(retryRequest)
                                              .willSetStateTo(retryRequestAgain)
                                              .willReturn(aResponse().withStatus(500)));

        wireMock.stubFor(get(urlEqualTo(path)).inScenario(retryScenario)
                                              .whenScenarioStateIs(retryRequestAgain)
                                              .willReturn(aResponse().withBody("OK")));
    }

    private static class CountingConnectionReuseStrategy extends ConfigServerRestExecutorImpl.ConnectionReuseStrategy {

        private final Map<String, Integer> reusedConnections = new HashMap<>();

        public CountingConnectionReuseStrategy(Set<String> vips) {
            super(vips);
        }

        @Override
        public boolean keepAlive(org.apache.http.HttpResponse response, HttpContext context) {
            boolean keepAlive = super.keepAlive(response, context);
            String host = HttpCoreContext.adapt(context).getTargetHost().getHostName();
            reusedConnections.putIfAbsent(host, 0);
            if (keepAlive) reusedConnections.compute(host, (ignored, count) -> ++count);
            return keepAlive;
        }

    }

}