query param should have a ? in trace (#49)

minio · Nov 19, 2020 · 18f3ddb · 18f3ddb
1 parent ce2ac61
commit 18f3ddb
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/http-tracer.go b/http-tracer.go
@@ -375,7 +375,11 @@ func (s shortTraceMsg) String() string {
 	fmt.Fprintf(b, "[%s] ", statusStr)
 	fmt.Fprintf(b, " %s ", s.Host)
 
-	fmt.Fprintf(b, "%s %s%s", s.Method, s.Path, s.Query)
+	if s.Query != "" {
+		fmt.Fprintf(b, "%s %s?%s", s.Method, s.Path, s.Query)
+	} else {
+		fmt.Fprintf(b, "%s %s", s.Method, s.Path)
+	}
 	fmt.Fprintf(b, " %s ", s.Client)
 	spaces := 15 - len(s.Client)
 	fmt.Fprintf(b, "%*s", spaces, " ")