Page MenuHomePhabricator
Differential D12335

[identity] reintroduce check to make sure only wallet users can call deleteWalletUser RPC
ClosedPublic
Actions

Authored by varun on Thu, Jun 6, 1:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 27, 4:49 AM
Unknown Object (File)
Wed, Jun 26, 9:24 AM
Unknown Object (File)
Tue, Jun 25, 9:40 PM
Unknown Object (File)
Tue, Jun 25, 4:13 AM
Unknown Object (File)
Mon, Jun 24, 1:07 PM
Unknown Object (File)
Sun, Jun 23, 10:31 AM
Unknown Object (File)
Fri, Jun 21, 5:44 AM
Unknown Object (File)
Wed, Jun 19, 2:09 PM
View All 24 Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
will
Commits
rCOMMd08a0d0edfbb: [identity] reintroduce check to make sure only wallet users can call…
Summary

Depends on D12326

Devs will have to pull changes from D12326 to delete password accounts after this change is added to staging

Test Plan

i got a permission denied error trying to call deleteWalletUser from a password account

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun created this revision.Thu, Jun 6, 1:20 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Jun 6, 1:20 PM
varun updated this revision to Diff 41073.Thu, Jun 6, 1:25 PM

also removing 2 lines from web that are not necessary

Harbormaster completed remote builds in B29499: Diff 41072.Thu, Jun 6, 1:38 PM
Harbormaster completed remote builds in B29500: Diff 41073.Thu, Jun 6, 1:41 PM
varun requested review of this revision.Thu, Jun 6, 1:41 PM
will accepted this revision.Thu, Jun 6, 3:58 PM
This revision is now accepted and ready to land.Thu, Jun 6, 3:58 PM
varun added a child revision: D12353: [native][lib] update useDeleteDiscardedIdentityAccount to throw.Fri, Jun 7, 9:11 AM
Closed by commit rCOMMd08a0d0edfbb: [identity] reintroduce check to make sure only wallet users can call… (authored by varun). · Explain WhyFri, Jun 7, 11:47 AM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMMd08a0d0edfbb: [identity] reintroduce check to make sure only wallet users can call….
varun mentioned this in D12515: [identity] fix wallet account deletion bug.Thu, Jun 20, 11:13 AM
varun mentioned this in rCOMM30437f95dcbf: [identity] fix wallet account deletion bug.Mon, Jun 24, 10:29 AM

Revision Contents
Changeset List

PathSize
services/
identity/
src/
grpc_services/
10 lines
web/
grpc/
2 lines

Diff 41118

View Options

services/identity/src/grpc_services/authenticated.rs

Loading...
View Options

web/grpc/identity-service-context-provider.react.js

Loading...