Merge pull request #136 from ChilliCream/get-request

Author: michaelstaib

src/AspNetCore.Tests/QueryMiddlewareTests.cs

@@ -56,6 +56,21 @@ public async Task HttpGet_BasicTest()
             Assert.Equal(Snapshot.Current(), Snapshot.New(result));
         }
 
+        [Fact]
+        public async Task HttpGet_ForwardToNextMiddleware()
+        {
+            // arrange
+            TestServer server = CreateTestServer();
+            string query = "{ basic { a } }";
+
+            // act
+            HttpResponseMessage message = await server.CreateClient()
+                .GetAsync($"http://localhost:5000/1234");
+
+            // assert
+            Assert.Equal(HttpStatusCode.NotFound, message.StatusCode);
+        }
+
         [Fact]
         public async Task HttpPost_WithScalarVariables()
         {

src/AspNetCore/GetRequest.cs

@@ -10,20 +10,22 @@ internal static class GetRequest
 
         internal static bool IsGet(this HttpRequest request)
         {
-            return request.Method.Equals(_getMethod, StringComparison.OrdinalIgnoreCase);
+            return request.Method.Equals(_getMethod, StringComparison.OrdinalIgnoreCase)
+                   && HasQueryParameter(request.HttpContext);
         }
 
         internal static QueryRequest ReadRequest(HttpContext context)
         {
-            string query = string.Empty;
-
-            if (context.Request.QueryString.HasValue &&
-                context.Request.Query.ContainsKey(_queryIdentifier))
+            return new QueryRequest()
             {
-                query = context.Request.Query[_queryIdentifier].ToString();
-            }
+                Query = context.Request.Query[_queryIdentifier].ToString()
+            };
+        }
 
-            return new QueryRequest() {Query = query};
+        private static bool HasQueryParameter(HttpContext context)
+        {
+            return context.Request.QueryString.HasValue &&
+                   context.Request.Query.ContainsKey(_queryIdentifier);
         }
     }
 }