diff options
author | Paul Hawke <paul.hawke@gmail.com> | 2013-12-12 06:06:25 -0600 |
---|---|---|
committer | Paul Hawke <paul.hawke@gmail.com> | 2013-12-12 06:06:25 -0600 |
commit | ef3be58ec24d3e155c8036f8806dba7172f61d0c (patch) | |
tree | e829db9643634d8320e5cbb3282ac2197759bbbe | |
parent | 68394ef81a80928d25e584475be59a2e937a5ae0 (diff) | |
download | nanohttpd-ef3be58ec24d3e155c8036f8806dba7172f61d0c.tar.gz |
Tests and documentation cleanup.
-rw-r--r-- | README.md | 76 | ||||
-rw-r--r-- | core/src/test/java/fi/iki/elonen/HttpGetRequestTest.java | 4 | ||||
-rw-r--r-- | core/src/test/java/fi/iki/elonen/HttpHeadRequestTest.java | 4 | ||||
-rw-r--r-- | core/src/test/java/fi/iki/elonen/HttpServerTest.java | 25 | ||||
-rw-r--r-- | samples/src/main/java/fi/iki/elonen/TempFilesServer.java | 2 | ||||
-rw-r--r-- | samples/src/main/java/fi/iki/elonen/debug/DebugServer.java (renamed from samples/src/main/java/fi/iki/elonen/DebugServer.java) | 35 |
6 files changed, 108 insertions, 38 deletions
@@ -77,33 +77,83 @@ Secondly, you can run the standalone *NanoHttpd Webserver*. Or, create your own class that extends `NanoHTTPD` and overrides one of the two flavors of the `serve()` method. For example: ```java +package fi.iki.elonen.debug; + +import fi.iki.elonen.NanoHTTPD; +import fi.iki.elonen.ServerRunner; + +import java.util.HashMap; +import java.util.List; +import java.util.Map; + public class DebugServer extends NanoHTTPD { - /** - * Constructs an HTTP server on given port. - */ public DebugServer() { super(8080); } - @Override - public Response serve(String uri, Method method, Map<String, String> header, Map<String, String> parms, Map<String, String> files) { + public static void main(String[] args) { + ServerRunner.run(DebugServer.class); + } + + @Override public Response serve(IHTTPSession session) { + Map<String, List<String>> decodedQueryParameters = + decodeParameters(session.getQueryParameterString()); + StringBuilder sb = new StringBuilder(); sb.append("<html>"); sb.append("<head><title>Debug Server</title></head>"); sb.append("<body>"); - sb.append("<h1>Response</h1>"); - sb.append("<p><blockquote><b>URI -</b> ").append(uri).append("<br />"); - sb.append("<b>Method -</b> ").append(method).append("</blockquote></p>"); - sb.append("<h3>Headers</h3><p><blockquote>").append(header).append("</blockquote></p>"); - sb.append("<h3>Parms</h3><p><blockquote>").append(parms).append("</blockquote></p>"); - sb.append("<h3>Files</h3><p><blockquote>").append(files).append("</blockquote></p>"); + sb.append("<h1>Debug Server</h1>"); + + sb.append("<p><blockquote><b>URI</b> = ").append( + String.valueOf(session.getUri())).append("<br />"); + + sb.append("<b>Method</b> = ").append( + String.valueOf(session.getMethod())).append("</blockquote></p>"); + + sb.append("<h3>Headers</h3><p><blockquote>"). + append(toString(session.getHeaders())).append("</blockquote></p>"); + + sb.append("<h3>Parms</h3><p><blockquote>"). + append(toString(session.getParms())).append("</blockquote></p>"); + + sb.append("<h3>Parms (multi values?)</h3><p><blockquote>"). + append(toString(decodedQueryParameters)).append("</blockquote></p>"); + + try { + Map<String, String> files = new HashMap<String, String>(); + session.parseBody(files); + sb.append("<h3>Files</h3><p><blockquote>"). + append(toString(files)).append("</blockquote></p>"); + } catch (Exception e) { + e.printStackTrace(); + } + sb.append("</body>"); sb.append("</html>"); return new Response(sb.toString()); } - public static void main(String[] args) { - ServerRunner.run(DebugServer.class); + private String toString(Map<String, ? extends Object> map) { + if (map.size() == 0) { + return ""; + } + return unsortedList(map); + } + + private String unsortedList(Map<String, ? extends Object> map) { + StringBuilder sb = new StringBuilder(); + sb.append("<ul>"); + for (Map.Entry entry : map.entrySet()) { + listItem(sb, entry); + } + sb.append("</ul>"); + return sb.toString(); + } + + private void listItem(StringBuilder sb, Map.Entry entry) { + sb.append("<li><code><b>").append(entry.getKey()). + append("</b> = ").append(entry.getValue()).append("</code></li>"); } } ``` diff --git a/core/src/test/java/fi/iki/elonen/HttpGetRequestTest.java b/core/src/test/java/fi/iki/elonen/HttpGetRequestTest.java index c914249..598e50b 100644 --- a/core/src/test/java/fi/iki/elonen/HttpGetRequestTest.java +++ b/core/src/test/java/fi/iki/elonen/HttpGetRequestTest.java @@ -108,7 +108,7 @@ public class HttpGetRequestTest extends HttpServerTest { @Test public void testDecodingParametersWithSingleValue() { invokeServer("GET " + URI + "?foo=bar&baz=zot HTTP/1.1"); - assertEquals("foo=bar&baz=zot", testServer.parms.get("NanoHttpd.QUERY_STRING")); + assertEquals("foo=bar&baz=zot", testServer.queryParameterString); assertTrue(testServer.decodedParamters.get("foo") instanceof List); assertEquals(1, testServer.decodedParamters.get("foo").size()); assertEquals("bar", testServer.decodedParamters.get("foo").get(0)); @@ -120,7 +120,7 @@ public class HttpGetRequestTest extends HttpServerTest { @Test public void testDecodingParametersWithSingleValueAndMissingValue() { invokeServer("GET " + URI + "?foo&baz=zot HTTP/1.1"); - assertEquals("foo&baz=zot", testServer.parms.get("NanoHttpd.QUERY_STRING")); + assertEquals("foo&baz=zot", testServer.queryParameterString); assertTrue(testServer.decodedParamters.get("foo") instanceof List); assertEquals(0, testServer.decodedParamters.get("foo").size()); assertTrue(testServer.decodedParamters.get("baz") instanceof List); diff --git a/core/src/test/java/fi/iki/elonen/HttpHeadRequestTest.java b/core/src/test/java/fi/iki/elonen/HttpHeadRequestTest.java index 6dc547f..1c5901f 100644 --- a/core/src/test/java/fi/iki/elonen/HttpHeadRequestTest.java +++ b/core/src/test/java/fi/iki/elonen/HttpHeadRequestTest.java @@ -95,7 +95,7 @@ public class HttpHeadRequestTest extends HttpServerTest { @Test public void testDecodingParametersWithSingleValue() { invokeServer("HEAD " + URI + "?foo=bar&baz=zot HTTP/1.1"); - assertEquals("foo=bar&baz=zot", testServer.parms.get("NanoHttpd.QUERY_STRING")); + assertEquals("foo=bar&baz=zot", testServer.queryParameterString); assertTrue(testServer.decodedParamters.get("foo") instanceof List); assertEquals(1, testServer.decodedParamters.get("foo").size()); assertEquals("bar", testServer.decodedParamters.get("foo").get(0)); @@ -107,7 +107,7 @@ public class HttpHeadRequestTest extends HttpServerTest { @Test public void testDecodingParametersWithSingleValueAndMissingValue() { invokeServer("HEAD " + URI + "?foo&baz=zot HTTP/1.1"); - assertEquals("foo&baz=zot", testServer.parms.get("NanoHttpd.QUERY_STRING")); + assertEquals("foo&baz=zot", testServer.queryParameterString); assertTrue(testServer.decodedParamters.get("foo") instanceof List); assertEquals(0, testServer.decodedParamters.get("foo").size()); assertTrue(testServer.decodedParamters.get("baz") instanceof List); diff --git a/core/src/test/java/fi/iki/elonen/HttpServerTest.java b/core/src/test/java/fi/iki/elonen/HttpServerTest.java index d9b99b3..ef209d1 100644 --- a/core/src/test/java/fi/iki/elonen/HttpServerTest.java +++ b/core/src/test/java/fi/iki/elonen/HttpServerTest.java @@ -7,6 +7,7 @@ import org.junit.Test; import java.io.*; import java.net.InetAddress; import java.util.ArrayList; +import java.util.HashMap; import java.util.List; import java.util.Map; @@ -102,6 +103,7 @@ public class HttpServerTest { public Map<String, String> files; public Map<String, List<String>> decodedParamters; public Map<String, List<String>> decodedParamtersFromParameter; + public String queryParameterString; public TestServer() { super(8192); @@ -115,15 +117,20 @@ public class HttpServerTest { return new HTTPSession(tempFileManager, inputStream, outputStream, inetAddress); } - @Override - public Response serve(String uri, Method method, Map<String, String> header, Map<String, String> parms, Map<String, String> files) { - this.uri = uri; - this.method = method; - this.header = header; - this.parms = parms; - this.files = files; - this.decodedParamtersFromParameter = decodeParameters(parms); - this.decodedParamters = decodeParameters(parms.get("NanoHttpd.QUERY_STRING")); + @Override public Response serve(IHTTPSession session) { + this.uri = session.getUri(); + this.method = session.getMethod(); + this.header = session.getHeaders(); + this.parms = session.getParms(); + this.files = new HashMap<String, String>(); + try { + session.parseBody(files); + } catch (Exception e) { + e.printStackTrace(); + } + queryParameterString = session.getQueryParameterString(); + this.decodedParamtersFromParameter = decodeParameters(queryParameterString); + this.decodedParamters = decodeParameters(session.getQueryParameterString()); return response; } diff --git a/samples/src/main/java/fi/iki/elonen/TempFilesServer.java b/samples/src/main/java/fi/iki/elonen/TempFilesServer.java index 84eb871..fb96911 100644 --- a/samples/src/main/java/fi/iki/elonen/TempFilesServer.java +++ b/samples/src/main/java/fi/iki/elonen/TempFilesServer.java @@ -1,5 +1,7 @@ package fi.iki.elonen; +import fi.iki.elonen.debug.DebugServer; + import java.util.ArrayList; import java.util.List; diff --git a/samples/src/main/java/fi/iki/elonen/DebugServer.java b/samples/src/main/java/fi/iki/elonen/debug/DebugServer.java index ae3fff1..1c83876 100644 --- a/samples/src/main/java/fi/iki/elonen/DebugServer.java +++ b/samples/src/main/java/fi/iki/elonen/debug/DebugServer.java @@ -1,7 +1,10 @@ -package fi.iki.elonen; +package fi.iki.elonen.debug; + +import fi.iki.elonen.NanoHTTPD; +import fi.iki.elonen.ServerRunner; -import java.io.IOException; import java.util.HashMap; +import java.util.List; import java.util.Map; public class DebugServer extends NanoHTTPD { @@ -14,32 +17,39 @@ public class DebugServer extends NanoHTTPD { } @Override public Response serve(IHTTPSession session) { + Map<String, List<String>> decodedQueryParameters = + decodeParameters(session.getQueryParameterString()); + StringBuilder sb = new StringBuilder(); sb.append("<html>"); sb.append("<head><title>Debug Server</title></head>"); sb.append("<body>"); sb.append("<h1>Debug Server</h1>"); - sb.append("<p><blockquote><b>URI</b> = ").append(String.valueOf(session.getUri())).append("<br />"); + sb.append("<p><blockquote><b>URI</b> = ").append( + String.valueOf(session.getUri())).append("<br />"); - sb.append("<b>Method</b> = ").append(String.valueOf(session.getMethod())).append("</blockquote></p>"); + sb.append("<b>Method</b> = ").append( + String.valueOf(session.getMethod())).append("</blockquote></p>"); - sb.append("<h3>Headers</h3><p><blockquote>").append(toString(session.getHeaders())).append("</blockquote></p>"); + sb.append("<h3>Headers</h3><p><blockquote>"). + append(toString(session.getHeaders())).append("</blockquote></p>"); - sb.append("<h3>Parms</h3><p><blockquote>").append(toString(session.getParms())).append("</blockquote></p>"); + sb.append("<h3>Parms</h3><p><blockquote>"). + append(toString(session.getParms())).append("</blockquote></p>"); sb.append("<h3>Parms (multi values?)</h3><p><blockquote>"). - append(toString(decodeParameters(session.getQueryParameterString()))).append("</blockquote></p>"); + append(toString(decodedQueryParameters)).append("</blockquote></p>"); try { Map<String, String> files = new HashMap<String, String>(); session.parseBody(files); - sb.append("<h3>Files</h3><p><blockquote>").append(toString(files)).append("</blockquote></p>"); - } catch (IOException e) { - e.printStackTrace(); - } catch (ResponseException e) { + sb.append("<h3>Files</h3><p><blockquote>"). + append(toString(files)).append("</blockquote></p>"); + } catch (Exception e) { e.printStackTrace(); } + sb.append("</body>"); sb.append("</html>"); return new Response(sb.toString()); @@ -63,6 +73,7 @@ public class DebugServer extends NanoHTTPD { } private void listItem(StringBuilder sb, Map.Entry entry) { - sb.append("<li><code><b>").append(entry.getKey()).append("</b> = ").append(entry.getValue()).append("</code></li>"); + sb.append("<li><code><b>").append(entry.getKey()). + append("</b> = ").append(entry.getValue()).append("</code></li>"); } } |