You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by mh...@apache.org on 2015/07/23 17:31:06 UTC

[11/16] incubator-tinkerpop git commit: Gremlin Server REST route ignores request for favicon.

Gremlin Server REST route ignores request for favicon.


Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/278743f4
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/278743f4
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/278743f4

Branch: refs/heads/master
Commit: 278743f4aaeb62978f55832ec5bdf56ea8f5928f
Parents: 08f364d
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Wed Jul 22 16:41:55 2015 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Wed Jul 22 16:41:55 2015 -0400

----------------------------------------------------------------------
 .../gremlin/server/handler/HttpGremlinEndpointHandler.java     | 6 ++++++
 1 file changed, 6 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/278743f4/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
----------------------------------------------------------------------
diff --git a/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java b/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
index 9e9b8ef..afdba1d 100644
--- a/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
+++ b/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
@@ -129,6 +129,12 @@ public class HttpGremlinEndpointHandler extends ChannelInboundHandlerAdapter {
         if (msg instanceof FullHttpRequest) {
             final FullHttpRequest req = (FullHttpRequest) msg;
 
+            if ("/favicon.ico".equals(req.getUri())) {
+                sendError(ctx, NOT_FOUND, "Gremlin Server doesn't have a favicon.ico");
+                ReferenceCountUtil.release(msg);
+                return;
+            }
+
             if (is100ContinueExpected(req)) {
                 ctx.write(new DefaultFullHttpResponse(HTTP_1_1, CONTINUE));
             }