From 68276159ca7f37250a81c7bb8c00cefde027a99f Mon Sep 17 00:00:00 2001
From: Matt Jaffee <matthew.jaffee@gmail.com>
Date: Fri, 10 Apr 2020 21:07:43 -0500
Subject: [PATCH] don't access req.Query before knowing req is a QueryRequest

---
 http/handler.go | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/http/handler.go b/http/handler.go
index 51ce25991..cecaa666e 100644
--- a/http/handler.go
+++ b/http/handler.go
@@ -270,10 +270,10 @@ func (h *Handler) collectStats(next http.Handler) http.Handler {
 		longQueryTime := h.api.LongQueryTime()
 		if longQueryTime > 0 && dur > longQueryTime {
 			queryRequest := r.Context().Value(contextKeyQueryRequest)
-			req, ok := queryRequest.(*pilosa.QueryRequest)
-			queryString := req.Query
-			if !ok {
-				queryString = ""
+
+			var queryString string
+			if req, ok := queryRequest.(*pilosa.QueryRequest); ok {
+				queryString = req.Query
 			}
 
 			h.logger.Printf("%s %s %v %s", r.Method, r.URL.String(), dur, queryString)