[identity] remove user ID from "no device list" log
AbandonedPublic
Actions

Authored by varun on Jul 19 2024, 10:39 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Apr 7, 7:36 PM

	Unknown Object (File)
	Tue, Mar 11, 10:48 AM

	Unknown Object (File)
	Mar 5 2025, 2:24 PM

	Unknown Object (File)
	Feb 28 2025, 1:17 AM

	Unknown Object (File)
	Feb 26 2025, 11:42 AM

	Unknown Object (File)
	Feb 26 2025, 9:13 AM

	Unknown Object (File)
	Feb 20 2025, 2:47 AM

	Unknown Object (File)
	Jan 19 2025, 6:18 PM

View All 44 Files

Subscribers

tomek

Details

Reviewers

bartek
will
ashoat

Summary

we don't want to leak user info in access logs

Test Plan

confirmed that the printed log no longer contains user_id

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Jul 19 2024, 10:39 PM

Herald added a subscriber: tomek. · View Herald TranscriptJul 19 2024, 10:39 PM

Harbormaster completed remote builds in B30536: Diff 42576.Jul 19 2024, 10:55 PM

varun requested review of this revision.Jul 19 2024, 10:55 PM

ashoat accepted this revision.Jul 19 2024, 10:58 PM

This revision is now accepted and ready to land.Jul 19 2024, 10:58 PM

abandoning in favor of D12819 and D12820

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

grpc_services/

authenticated.rs

2 lines

Diff 42576

View Options

services/identity/src/grpc_services/authenticated.rs

[identity] remove user ID from "no device list" logAbandonedPublicActions