Page MenuHomePhabricator

[identity] Delete Backup user data on account deletion
ClosedPublic

Authored by bartek on Aug 5 2024, 4:54 AM.
Tags
None
Referenced Files
F3204811: D12973.id43095.diff
Sat, Nov 9, 9:59 PM
F3204770: D12973.id43090.diff
Sat, Nov 9, 9:52 PM
F3203191: D12973.diff
Sat, Nov 9, 7:25 PM
Unknown Object (File)
Wed, Nov 6, 10:49 PM
Unknown Object (File)
Wed, Nov 6, 10:49 PM
Unknown Object (File)
Wed, Nov 6, 10:49 PM
Unknown Object (File)
Wed, Nov 6, 10:49 PM
Unknown Object (File)
Wed, Nov 6, 10:19 PM
Subscribers

Details

Summary

Similarly to how we call Tunnelbroker, called Backup Service DELETE /user_data endpoint from D12970 to delete backup data
in DeleteUser RPCs.

Depends on D12972

Test Plan

Called the DeleteWalletUser RPC. Verified that Backup is called and user data is actually deleted from Backup DDB

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.

Improve service-to-service token handling

bartek published this revision for review.Aug 5 2024, 5:35 AM
bartek added inline comments.
services/identity/src/main.rs
88–89 ↗(On Diff #43095)

The second argument is identity service URL, but in fact, we're never going to need it, so I think it can stay hardcoded

This revision is now accepted and ready to land.Aug 5 2024, 6:10 AM
This revision was landed with ongoing or failed builds.Aug 5 2024, 6:28 AM
This revision was automatically updated to reflect the committed changes.