Added LogAllErrors config parameter to Server, which allows logging the most frequent errors such as 'connection reset by peer', 'broken pipe' and 'i/o timeout'. By default such errors are suppressed

1 files changed, 5 insertions, 4 deletions

diff --git a/workerpool.go b/workerpool.go
index 931ab35..03aee88 100644
--- a/workerpool.go
+++ b/workerpool.go
@@ -19,10 +19,10 @@ type workerPool struct {
 	// It must leave c unclosed.
 	WorkerFunc func(c net.Conn) error
 
-	// Maximum number of workers to create.
 	MaxWorkersCount int
 
-	// Logger used by workerPool.
+	LogAllErrors bool
+
 	Logger Logger
 
 	lock         sync.Mutex
@@ -194,8 +194,9 @@ func (wp *workerPool) workerFunc(ch *workerChan) {
 		}
 		if err = wp.WorkerFunc(c); err != nil && err != errHijacked {
 			errStr := err.Error()
-			if !strings.Contains(errStr, "broken pipe") && !strings.Contains(errStr, "reset by peer") &&
-				!strings.Contains(errStr, "i/o timeout") {
+			if wp.LogAllErrors || !(strings.Contains(errStr, "broken pipe") ||
+				strings.Contains(errStr, "reset by peer") ||
+				strings.Contains(errStr, "i/o timeout")) {
 				wp.Logger.Printf("error when serving connection %q<->%q: %s", c.LocalAddr(), c.RemoteAddr(), err)
 			}
 		}