TrueCloudLab
/
frostfs-s3-gw
15
2
Fork 15
Code Issues 20 Pull Requests 6 Actions Packages Projects Releases 7 Activity

[#396] Add user to response #396

Merged
alexvanin merged 1 commits from pogpp/frostfs-s3-gw:feature/add_logs_to_request into master 2024-06-04 09:37:56 +00:00
Conversation 0 Commits 1 Files Changed 2
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
api/handler/util.go
View File

@ -42,6 +42,7 @@ func (h *handler) logAndSendError(w http.ResponseWriter, logText string, reqInfo
zap.String("bucket", reqInfo.BucketName),
zap.String("object", reqInfo.ObjectName),
zap.String("description", logText),
zap.String("user", reqInfo.User),
zap.Error(err)}
fields = append(fields, additional...)
if traceID, err := trace.TraceIDFromHex(reqInfo.TraceID); err == nil && traceID.IsValid() {

3
api/middleware/response.go
View File

@ -328,6 +328,9 @@ func LogSuccessResponse(l *zap.Logger) Func {
if reqInfo.ObjectName != "" {
fields = append(fields, zap.String("object", reqInfo.ObjectName))
}
if reqInfo.User != "" {
fields = append(fields, zap.String("user", reqInfo.User))
dkirillov marked this conversation as resolved Outdated
dkirillov commented 2024-05-28 06:21:43 +00:00
Outdated
Review

Do we need the same when request ends with error?

Do we need the same when request ends with error?
pogpp commented 2024-05-28 11:53:10 +00:00
Outdated
Review

Do we need the same when request ends with error?

Agree - added

> Do we need the same when request ends with error? Agree - added
dkirillov commented 2024-05-29 06:59:30 +00:00
Outdated
Review

I meant just log user in case of request failed (somewhere here) rather than include user info into error response.

I meant just log user in case of request failed (somewhere [here](https://git.frostfs.info/TrueCloudLab/frostfs-s3-gw/src/commit/9787b295427034396ed2ead8d0c1687a25beb4ec/api/handler/util.go#L39)) rather than include user info into error response.
👍 2
pogpp commented 2024-05-29 12:30:14 +00:00
Outdated
Review

done

done
}
if traceID, err := trace.TraceIDFromHex(reqInfo.TraceID); err == nil && traceID.IsValid() {
fields = append(fields, zap.String("trace_id", reqInfo.TraceID))