Skip to content

Commit 48c3482

Browse files
committed
don't log requests
1 parent a210eea commit 48c3482

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

recovery.go

+1-3
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import (
1111
"io/ioutil"
1212
"log"
1313
"net/http"
14-
"net/http/httputil"
1514
"runtime"
1615
"time"
1716
)
@@ -39,8 +38,7 @@ func RecoveryWithWriter(out io.Writer) HandlerFunc {
3938
if err := recover(); err != nil {
4039
if logger != nil {
4140
stack := stack(3)
42-
httprequest, _ := httputil.DumpRequest(c.Request, false)
43-
logger.Printf("[Recovery] %s panic recovered:\n%s\n%s\n%s%s", timeFormat(time.Now()), string(httprequest), err, stack, reset)
41+
logger.Printf("[Recovery] %s panic recovered:\n%s\n%s%s", timeFormat(time.Now()), err, stack, reset)
4442
}
4543
c.AbortWithStatus(http.StatusInternalServerError)
4644
}

recovery_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ func TestPanicInHandler(t *testing.T) {
2424
w := performRequest(router, "GET", "/recovery")
2525
// TEST
2626
assert.Equal(t, http.StatusInternalServerError, w.Code)
27-
assert.Contains(t, buffer.String(), "GET /recovery")
27+
assert.Contains(t, buffer.String(), "panic recovered")
2828
assert.Contains(t, buffer.String(), "Oupps, Houston, we have a problem")
2929
assert.Contains(t, buffer.String(), "TestPanicInHandler")
3030
}

0 commit comments

Comments
 (0)