Differential D11796

[identity] delete all token data on user account deletion
ClosedPublic
Actions

Authored by varun on Apr 25 2024, 4:45 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Nov 13, 7:57 PM

	Unknown Object (File)
	Wed, Nov 13, 12:50 PM

	Unknown Object (File)
	Tue, Nov 12, 4:21 PM

	Unknown Object (File)
	Tue, Nov 12, 3:18 PM

	Unknown Object (File)
	Oct 14 2024, 5:32 PM

	Unknown Object (File)
	Oct 11 2024, 7:48 AM

	Unknown Object (File)
	Oct 11 2024, 7:48 AM

	Unknown Object (File)
	Oct 11 2024, 7:48 AM

View All 49 Files

Subscribers

Details

Reviewers

Commits

rCOMMb08ab7a0087b: [identity] delete all token data on user account deletion

Summary

new db method to get all tokens for a user and then delete them with BatchWriteItem. we use the comm-lib batch_write function for the delete requests.

Depends on D11795

Test Plan

signed in to a local account from web, native, and keyserver. then deleted my account from native. confirmed that there were no tokens in local ddb for my user ID

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Apr 25 2024, 4:45 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptApr 25 2024, 4:45 PM

Harbormaster completed remote builds in B28492: Diff 39522.Apr 25 2024, 5:00 PM

varun requested review of this revision.Apr 25 2024, 5:00 PM

bartek accepted this revision.Apr 26 2024, 1:14 AM

bartek added inline comments.

services/identity/src/constants.rs
61–69 ↗	(On Diff #39522)	👍

This revision is now accepted and ready to land.Apr 26 2024, 1:14 AM

Closed by commit rCOMMb08ab7a0087b: [identity] delete all token data on user account deletion (authored by varun). · Explain WhyApr 26 2024, 11:31 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMb08ab7a0087b: [identity] delete all token data on user account deletion.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

16 lines

3 lines

database/

138 lines

Diff 39572

services/identity/src/constants.rs

Loading...

services/identity/src/database.rs

Loading...

services/identity/src/database/token.rs

Loading...