storj / edge

Storj edge services (including multi-tenant, S3-compatible server to interact with the Storj network)
GNU Affero General Public License v3.0
48 stars 18 forks source link

Logging fixes: 501 should be a warning, don't log 4XXs #387

Closed amwolff closed 5 months ago

amwolff commented 7 months ago

Goal

4XXs are a normal operation and are cluttering the INFO level.

This is also the case with 501s ("Not Implemented") on the ERROR level.

Acceptance Criteria

### Tasks
- [ ] Move 4XXs to DEBUG
- [ ] Move 501s to WARNING
storj-gerrit[bot] commented 5 months ago

Change pkg/httplog: change log levels for 4xx and 501 status mentions this issue.

storj-gerrit[bot] commented 5 months ago

Change pkg/server/middleware: add additional fields to gmt events mentions this issue.

storj-gerrit[bot] commented 5 months ago

Change {pkg/httplog,pkg/server/middleware}: move common log code to httplog mentions this issue.