don't access req.Query before knowing req is a QueryRequest

This commit is contained in:
Matt Jaffee 2020-04-10 21:07:43 -05:00
parent 18c6d8f76f
commit 68276159ca
No known key found for this signature in database
GPG key ID: 08A3DFFF987B11BF

View file

@ -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)