Paths

Table of Contentst

[lib] Sanitize Redux `CryptoStore`
ClosedPublic
Actions

Authored by atul on Mar 2 2023, 1:18 PM.

Tags

None

Referenced Files

	F3702421: D6931.id23377.diff
	Tue, Jan 7, 6:47 PM

	F3702420: D6931.id23366.diff
	Tue, Jan 7, 6:47 PM

	F3702416: D6931.id.diff
	Tue, Jan 7, 6:47 PM

	F3702408: D6931.diff
	Tue, Jan 7, 6:42 PM

	Unknown Object (File)
	Mon, Jan 6, 7:47 PM

	Unknown Object (File)
	Oct 28 2024, 11:19 PM

	Unknown Object (File)
	Oct 28 2024, 11:17 PM

	Unknown Object (File)
	Oct 20 2024, 3:24 AM

View All 72 Files

Subscribers

None

Details

Reviewers

ashoat
tomek
marcin

Commits

rCOMM807b989e5ef9: [lib] Sanitize Redux `CryptoStore`

Summary

Add keys to keysWithStringsToBeRedacted to properly redact CryptoStore.

Test Plan

Checked Redux dev tools to make sure I got the keys right:

Ran sanitizeAction and sanitizeState on the CryptoStore actions and state afterwards and ensured things were as expected:

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Mar 2 2023, 1:18 PM

atul edited the test plan for this revision. (Show Details)Mar 2 2023, 1:26 PM

atul edited the test plan for this revision. (Show Details)Mar 2 2023, 1:29 PM

Harbormaster completed remote builds in B17064: Diff 23366.Mar 2 2023, 1:32 PM

atul requested review of this revision.Mar 2 2023, 1:32 PM

atul edited the test plan for this revision. (Show Details)Mar 2 2023, 1:42 PM

ashoat accepted this revision.Mar 2 2023, 1:46 PM

This revision is now accepted and ready to land.Mar 2 2023, 1:46 PM

Closed by commit rCOMM807b989e5ef9: [lib] Sanitize Redux `CryptoStore` (authored by atul). · Explain WhyMar 2 2023, 2:17 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM807b989e5ef9: [lib] Sanitize Redux `CryptoStore`.

Revision Contents
Changeset List

Path

Size

lib/

utils/

sanitization.js

4 lines

Diff 23377

View Options

[lib] Sanitize Redux `CryptoStore`ClosedPublicActions