Set "Not Found" log-line to debug level (#1335)

404 is an expected signal during normal operation thus should not be logged as application error.
cs3org · Nov 26, 2020 · 1c9d29e · 1c9d29e
1 parent 27afb32
commit 1c9d29e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/rhttp/datatx/manager/simple/simple.go b/pkg/rhttp/datatx/manager/simple/simple.go
@@ -81,7 +81,7 @@ func (m *manager) Handler(fs storage.FS) (http.Handler, error) {
 			rc, err := fs.Download(ctx, ref)
 			if err != nil {
 				if _, ok := err.(errtypes.IsNotFound); ok {
-					log.Err(err).Msg("datasvc: file not found")
+					log.Debug().Err(err).Msg("datasvc: file not found")
 					w.WriteHeader(http.StatusNotFound)
 				} else {
 					log.Err(err).Msg("datasvc: error downloading file")

diff --git a/pkg/rhttp/datatx/manager/tus/tus.go b/pkg/rhttp/datatx/manager/tus/tus.go
@@ -123,7 +123,7 @@ func (m *manager) Handler(fs storage.FS) (http.Handler, error) {
 			rc, err := fs.Download(ctx, ref)
 			if err != nil {
 				if _, ok := err.(errtypes.IsNotFound); ok {
-					log.Err(err).Msg("datasvc: file not found")
+					log.Debug().Err(err).Msg("datasvc: file not found")
 					w.WriteHeader(http.StatusNotFound)
 				} else {
 					log.Err(err).Msg("datasvc: error downloading file")