Publish

1 files changed, 165 insertions, 0 deletions

diff --git a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/ConnectorFactoryTest.java b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/ConnectorFactoryTest.java
new file mode 100644
index 00000000000..425c0444252
--- /dev/null
+++ b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/ConnectorFactoryTest.java
@@ -0,0 +1,165 @@
+// Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
+package com.yahoo.jdisc.http.server.jetty;
+
+import com.yahoo.jdisc.Metric;
+import com.yahoo.jdisc.http.CertificateStore;
+import com.yahoo.jdisc.http.ConnectorConfig;
+import com.yahoo.jdisc.http.HttpRequest;
+import com.yahoo.jdisc.http.SecretStore;
+import com.yahoo.jdisc.http.ssl.ReaderForPath;
+import com.yahoo.jdisc.http.ssl.SslKeyStore;
+import com.yahoo.jdisc.http.ssl.SslKeyStoreFactory;
+import org.eclipse.jetty.server.Request;
+import org.eclipse.jetty.server.Server;
+import org.eclipse.jetty.server.handler.AbstractHandler;
+import org.testng.annotations.Test;
+
+import javax.servlet.ServletException;
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+import java.io.IOException;
+import java.net.InetSocketAddress;
+import java.nio.channels.ServerSocketChannel;
+import java.util.Collections;
+import java.util.Map;
+
+import static com.yahoo.jdisc.http.ConnectorConfig.*;
+import static com.yahoo.jdisc.http.ConnectorConfig.Ssl.KeyStoreType.Enum.JKS;
+import static com.yahoo.jdisc.http.ConnectorConfig.Ssl.KeyStoreType.Enum.PEM;
+import static org.hamcrest.CoreMatchers.equalTo;
+
+/**
+ * @author <a href="mailto:einarmr@yahoo-inc.com">Einar M R Rosenvinge</a>
+ */
+public class ConnectorFactoryTest {
+
+    @Test(expectedExceptions = IllegalArgumentException.class)
+    public void ssl_jks_config_is_validated() {
+        ConnectorConfig config = new ConnectorConfig(
+                new ConnectorConfig.Builder()
+                        .ssl(new Ssl.Builder()
+                                     .enabled(true)
+                                     .keyStoreType(JKS)
+                                     .pemKeyStore(
+                                             new Ssl.PemKeyStore.Builder()
+                                                     .keyPath("nonEmpty"))));
+
+        ConnectorFactory willThrowException = new ConnectorFactory(config, new ThrowingSslKeyStoreFactory(),
+                                                                   new ThrowingSecretStore());
+    }
+
+    @Test(expectedExceptions = IllegalArgumentException.class)
+    public void ssl_pem_config_is_validated() {
+        ConnectorConfig config = new ConnectorConfig(
+                new ConnectorConfig.Builder()
+                        .ssl(new Ssl.Builder()
+                                     .enabled(true)
+                                     .keyStoreType(PEM)
+                                     .keyStorePath("nonEmpty")));
+
+        ConnectorFactory willThrowException = new ConnectorFactory(config, new ThrowingSslKeyStoreFactory(),
+                                                                   new ThrowingSecretStore());
+    }
+
+    @Test
+    public void requireThatNoPreBoundChannelWorks() throws Exception {
+        Server server = new Server();
+        try {
+            ConnectorFactory factory = new ConnectorFactory(new ConnectorConfig(new ConnectorConfig.Builder()),
+                                                            new ThrowingSslKeyStoreFactory(),
+                                                            new ThrowingSecretStore());
+            ConnectorFactory.JDiscServerConnector connector =
+                    (ConnectorFactory.JDiscServerConnector)factory.createConnector(new DummyMetric(), server, null, Collections.emptyMap());
+            server.addConnector(connector);
+            server.setHandler(new HelloWorldHandler());
+            server.start();
+
+            SimpleHttpClient client = new SimpleHttpClient(null, connector.getLocalPort(), false);
+            SimpleHttpClient.RequestExecutor ex = client.newGet("/blaasdfnb");
+            SimpleHttpClient.ResponseValidator val = ex.execute();
+            val.expectContent(equalTo("Hello world"));
+        } finally {
+            try {
+                server.stop();
+            } catch (Exception e) {
+                //ignore
+            }
+        }
+    }
+
+    @Test
+    public void requireThatPreBoundChannelWorks() throws Exception {
+        Server server = new Server();
+        try {
+            ServerSocketChannel serverChannel = ServerSocketChannel.open();
+            serverChannel.socket().bind(new InetSocketAddress(0));
+
+            ConnectorFactory factory = new ConnectorFactory(new ConnectorConfig(new ConnectorConfig.Builder()), new ThrowingSslKeyStoreFactory(), new ThrowingSecretStore());
+            ConnectorFactory.JDiscServerConnector connector = (ConnectorFactory.JDiscServerConnector) factory.createConnector(new DummyMetric(), server, serverChannel, Collections.emptyMap());
+            server.addConnector(connector);
+            server.setHandler(new HelloWorldHandler());
+            server.start();
+
+            SimpleHttpClient client = new SimpleHttpClient(null, connector.getLocalPort(), false);
+            SimpleHttpClient.RequestExecutor ex = client.newGet("/blaasdfnb");
+            SimpleHttpClient.ResponseValidator val = ex.execute();
+            val.expectContent(equalTo("Hello world"));
+        } finally {
+            try {
+                server.stop();
+            } catch (Exception e) {
+                //ignore
+            }
+        }
+    }
+
+    private static class HelloWorldHandler extends AbstractHandler {
+        @Override
+        public void handle(String target, Request baseRequest, HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException {
+            response.getWriter().write("Hello world");
+            response.getWriter().flush();
+            response.getWriter().close();
+            baseRequest.setHandled(true);
+        }
+    }
+
+    private static class DummyMetric implements Metric {
+        @Override
+        public void set(String key, Number val, Context ctx) { }
+
+        @Override
+        public void add(String key, Number val, Context ctx) { }
+
+        @Override
+        public Context createContext(Map<String, ?> properties) {
+            return new DummyContext();
+        }
+    }
+
+    private static class DummyContext implements Metric.Context {
+    }
+
+    private static final class ThrowingSslKeyStoreFactory implements SslKeyStoreFactory {
+
+        @Override
+        public SslKeyStore createKeyStore(ReaderForPath certificateFile, ReaderForPath keyFile) {
+            throw new UnsupportedOperationException("A SSL key store factory component is not available");
+        }
+
+        @Override
+        public SslKeyStore createTrustStore(ReaderForPath certificateFile) {
+            throw new UnsupportedOperationException("A SSL key store factory component is not available");
+        }
+
+    }
+
+    private static final class ThrowingSecretStore implements SecretStore {
+
+        @Override
+        public String getSecret(String key) {
+            throw new UnsupportedOperationException("A secret store is not available");
+        }
+
+    }
+
+}