Page MenuHomePhabricator
Differential D12830

[identity] check users table before reserved usernames table for user ID during wallet login
ClosedPublic
Actions

Authored by varun on Jul 21 2024, 12:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 6:09 AM
Unknown Object (File)
Fri, Nov 1, 10:57 AM
Unknown Object (File)
Fri, Nov 1, 10:57 AM
Unknown Object (File)
Fri, Nov 1, 10:57 AM
Unknown Object (File)
Fri, Nov 1, 10:57 AM
Unknown Object (File)
Oct 24 2024, 6:18 AM
Unknown Object (File)
Oct 21 2024, 10:32 PM
Unknown Object (File)
Oct 16 2024, 7:47 PM
View All 33 Files
Subscribers
ashoat
tomek

Details

Reviewers
will
bartek
Commits
rCOMM63e8e9dea808: [identity] check users table before reserved usernames table for user ID during…
Summary

we should check the users table first in case the user ID happens to be in both tables for some reason (this should never happen, but unfortunately there was a bug that led to this rule being violated). by checking the users table first, we ensure that registered wallet users will be able to log in even if there is invalid data in the reserved usernames table.

Test Plan

add userID and wallet address to staging reserved usernames table. then successfully log in with wallet

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun created this revision.Jul 21 2024, 12:04 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 21 2024, 12:04 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Jul 21 2024, 12:15 PM
Harbormaster failed remote builds in B30550: Diff 42597!
varun requested review of this revision.Jul 21 2024, 1:38 PM
bartek accepted this revision.Jul 22 2024, 1:50 AM
bartek added inline comments.
services/identity/src/client_service.rs
521 ↗(On Diff #42597)

you can use login_time here too

This revision is now accepted and ready to land.Jul 22 2024, 1:50 AM
This revision was landed with ongoing or failed builds.Jul 22 2024, 10:03 AM
Closed by commit rCOMM63e8e9dea808: [identity] check users table before reserved usernames table for user ID during… (authored by varun). · Explain Why
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMM63e8e9dea808: [identity] check users table before reserved usernames table for user ID during….

Revision Contents
Changeset List

PathSize
services/
identity/
src/
52 lines

Diff 42631

View Options

services/identity/src/client_service.rs

Loading...