commit e543c51053e9c700e8175794941ba5b68ca42eda
author Edwin Kempin <ekempin@google.com> Wed Oct 04 13:23:14 2017 +0200
committer Edwin Kempin <ekempin@google.com> Wed Oct 04 13:23:14 2017 +0200
tree c5bed7cdcc54d9780b65a9bcb5010f2c39f12967
parent 33b527faf0be4c64f4c8de01f6b06ca8d9f5219b

Log invalid change queries on debug level

This allows us to investigate change query requests that were rejected
with 400 Bad Request.

Change-Id: Ie27c2b127bbb457ccda45611da302bd6a3e867a1
Signed-off-by: Edwin Kempin <ekempin@google.com>

gerrit-server/src/main/java/com/google/gerrit/server/query/change/QueryChanges.java

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/query/change/QueryChanges.java b/gerrit-server/src/main/java/com/google/gerrit/server/query/change/QueryChanges.java
index 65de137..c7a525a 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/query/change/QueryChanges.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/query/change/QueryChanges.java
@@ -35,8 +35,12 @@
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 import org.kohsuke.args4j.Option;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class QueryChanges implements RestReadView<TopLevelResource> {
+  private static final Logger log = LoggerFactory.getLogger(QueryChanges.class);
+
   private final ChangeJson.Factory json;
   private final ChangeQueryBuilder qb;
   private final ChangeQueryProcessor imp;
@@ -115,6 +119,7 @@
         String op = m.group(1);
         throw new AuthException("Must be signed-in to use " + op);
       }
+      log.debug("Reject change query with 400 Bad Request: " + queries, e);
       throw new BadRequestException(e.getMessage(), e);
     }
     return out.size() == 1 ? out.get(0) : out;