[blob] Don't log request payload in tracing span
ClosedPublic
Actions

Authored by bartek on Mon, Sep 30, 4:34 AM.

Tags

None

Referenced Files

	F2983611: D13516.diff
	Wed, Oct 16, 9:18 AM

	Unknown Object (File)
	Sun, Oct 13, 9:34 AM

	Unknown Object (File)
	Sun, Oct 13, 2:15 AM

	Unknown Object (File)
	Sat, Oct 12, 2:47 AM

	Unknown Object (File)
	Thu, Oct 10, 2:07 PM

	Unknown Object (File)
	Wed, Oct 9, 4:21 PM

	Unknown Object (File)
	Tue, Oct 8, 3:35 PM

	Unknown Object (File)
	Tue, Oct 8, 8:15 AM

View All 26 Files

Subscribers

ashoat

tomek

Details

Reviewers

will
varun
kamil

Commits

rCOMMc9f0d1d0a045: [blob] Don't log request payload in tracing span

Summary

Noticed that there's some spam in blob logs because the instrument includes full request payload in logs

Test Plan

Ran local Blob Service, called the endpoint. The payload is no longer printed.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Mon, Sep 30, 4:34 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMon, Sep 30, 4:34 AM

Harbormaster completed remote builds in B31924: Diff 44669.Mon, Sep 30, 5:06 AM

bartek published this revision for review.Mon, Sep 30, 5:54 AM

kamil accepted this revision.Mon, Sep 30, 6:35 AM

This revision is now accepted and ready to land.Mon, Sep 30, 6:35 AM

Closed by commit rCOMMc9f0d1d0a045: [blob] Don't log request payload in tracing span (authored by bartek). · Explain WhyTue, Oct 1, 6:44 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMc9f0d1d0a045: [blob] Don't log request payload in tracing span.

Revision Contents
Changeset List

Path

Size

services/

blob/

src/

http/

handlers/

holders.rs

2 lines

Diff 44669

View Options

services/blob/src/http/handlers/holders.rs

[blob] Don't log request payload in tracing spanClosedPublicActions