Merge pull request jetty#12483 from jetty/jetty-12.0.x-12482-CustomRequestLogQuery

PR jetty#12482 - fix bug with CustomRequestLog query string

Author: lachlan-roberts

jetty-core/jetty-server/src/main/java/org/eclipse/jetty/server/CustomRequestLog.java

@@ -1092,7 +1092,8 @@ private static void logResponseHeader(String arg, StringBuilder b, Request reque
     @SuppressWarnings("unused")
     private static void logQueryString(StringBuilder b, Request request, Response response)
     {
-        append(b, "?" + request.getHttpURI().getQuery());
+        String query = request.getHttpURI().getQuery();
+        append(b, (query == null) ? null : "?" + query);
     }
 
     @SuppressWarnings("unused")

jetty-core/jetty-server/src/test/java/org/eclipse/jetty/server/CustomRequestLogTest.java

@@ -502,6 +502,17 @@ public void testLogQueryString() throws Exception
         assertThat(log, is("QueryString: ?queryString"));
     }
 
+    @Test
+    public void testLogEmptyQueryString() throws Exception
+    {
+        start("QueryString: %q");
+
+        HttpTester.Response response = getResponse("GET /path HTTP/1.0\n\n");
+        assertEquals(HttpStatus.OK_200, response.getStatus());
+        String log = _logs.poll(5, TimeUnit.SECONDS);
+        assertThat(log, is("QueryString: -"));
+    }
+
     @Test
     public void testLogRequestFirstLine() throws Exception
     {