From 18f3ddbb60a7d68b6d12b4927a7015703e1aaa8d Mon Sep 17 00:00:00 2001
From: Harshavardhana <harsha@minio.io>
Date: Thu, 19 Nov 2020 03:23:03 -0800
Subject: [PATCH] query param should have a ? in trace (#49)

---
 http-tracer.go | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/http-tracer.go b/http-tracer.go
index 56999da..4d71204 100644
--- 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, " ")