Paths

Table of Contentst

[lib] Add more keys to redacted values
ClosedPublic
Actions

Authored by bartek on Sep 9 2024, 6:09 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Nov 7, 1:36 PM

	Unknown Object (File)
	Tue, Oct 22, 10:45 AM

	Unknown Object (File)
	Tue, Oct 22, 10:19 AM

	Unknown Object (File)
	Tue, Oct 22, 10:19 AM

	Unknown Object (File)
	Mon, Oct 14, 2:28 PM

	Unknown Object (File)
	Oct 4 2024, 7:59 PM

	Unknown Object (File)
	Sep 29 2024, 8:22 PM

	Unknown Object (File)
	Sep 29 2024, 6:18 AM

View All 30 Files

Subscribers

tomek

Details

Reviewers

ashoat

Commits

rCOMM5622ed8fcc83: [lib] Add more keys to redacted values

Summary

Address ENG-8717.

Added more keys to the redacted lists: CSAT, device lists, encrypted blob data
For Blob URIs, created a separate placeholder value

Test Plan

Triggered crash report on my test account. Verified that some Redux values (like commServicesAccessToken) are now redacted.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Sep 9 2024, 6:09 AM

bartek held this revision as a draft.

Herald added a subscriber: tomek. · View Herald TranscriptSep 9 2024, 6:09 AM

Harbormaster completed remote builds in B31494: Diff 43980.Sep 9 2024, 6:38 AM

bartek published this revision for review.Sep 9 2024, 6:40 AM

Thanks for doing this!

This revision is now accepted and ready to land.Sep 9 2024, 1:40 PM

Closed by commit rCOMM5622ed8fcc83: [lib] Add more keys to redacted values (authored by bartek). · Explain WhySep 11 2024, 12:53 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM5622ed8fcc83: [lib] Add more keys to redacted values.

Revision Contents
Changeset List

Path

Size

lib/

utils/

sanitization.js

18 lines

Diff 44021

View Options

[lib] Add more keys to redacted valuesClosedPublicActions