admin-server uses `ctx.Logger` but not initialized

journeymidnight / yig

An open source object storage server with Amazon S3 compatible API

Apache License 2.0

361 stars 86 forks source link

admin-server uses `ctx.Logger` but not initialized #292

Open cannium opened 4 years ago

cannium commented 4 years ago

On error, admin-server would use WriteErrorResponse, which relies on r.Context().Value(RequestContextKey).(RequestContext), but admin-server doesn't have such middleware initialized.