Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(Switchboard): DID user authentication and registration prototype #2497

Draft
wants to merge 45 commits into
base: master
Choose a base branch
from
Draft

feat(Switchboard): DID user authentication and registration prototype #2497

wants to merge 45 commits into from

Conversation

artursudnik
Copy link
Member

@artursudnik artursudnik commented Jun 8, 2021 •
edited
Loading

To be implemented:

  • tests
  • auth/login-did endpoint integrated with passport-did-auth
  • user registration form that makes use of the user/register-did endpoint
  • add other wallets choice on user-login page
  • user/did-roles - endpoint to get DID user roles on frontend

artursudnik added 14 commits June 8, 2021 10:27
@artursudnik
adjusted IJWTPayload
2ff018f
@artursudnik
auth/login-did endpoint added handled temporarily with 'did' login st…
17f99eb 
…rategy, populating access token with verifiedRoles based on actual db user rights field for testing
@artursudnik
jwt passport strategy populates user.rights if access token contains …
cd2b51c 
…verifiedRoles field
@artursudnik
filter out DID token roles not listed in the process.env.ACCEPTED_ROLES
0c7ddc2
@artursudnik
synchronize user.rights record to DID access token roles
df2dd5b
@artursudnik
set user.password as nullable
7c6e09e
@artursudnik
extend User entity with a nullable did string field
c9326a0
@artursudnik
added user/register-did endpoint
c9c7fab
@artursudnik
jwt-basic passport strategy used to protect user/register-did endpoint
73d601b
@artursudnik
UI: added iam-client-lib dependency
8bd089d
@artursudnik
iam-client-lib upgraded to 3.0.0-alpha.6
768b530
@artursudnik
frontend iam-client-lib integration
6e5d461
@artursudnik
"Login with DID and Metamask" button prototype
e14a361
@artursudnik
upgraded iam-client-lib to 3.0.0-alpha.17
0db4845
@artursudnik artursudnik marked this pull request as draft June 8, 2021 08:59
@artursudnik artursudnik changed the title feat(Switchboard): User authentication and registration prototype feat(Switchboard): DID user authentication and registration prototype Jun 8, 2021
@josipbagaric
Copy link
Contributor

Not relevant yet, but just a reminder before we merge.

We should re-write the commit history of this PR to comply to conventional commits before we merge it (see other PRs for examples).

Conventional commit messages are used to auto-generate release logs, hence super important.

@artursudnik artursudnik force-pushed the feat/sb-integration-prototype-user branch 3 times, most recently from 2b5149a to b8a733d Compare June 14, 2021 11:08
@artursudnik artursudnik force-pushed the feat/sb-integration-prototype-user branch from 0a18c1c to 74c14c5 Compare June 15, 2021 11:20
@artursudnik artursudnik force-pushed the feat/sb-integration-prototype-user branch from 9c46cac to 2aff636 Compare June 15, 2021 15:44
@artursudnik artursudnik force-pushed the feat/sb-integration-prototype-user branch from 2aff636 to fee9438 Compare June 16, 2021 07:44
@artursudnik artursudnik force-pushed the feat/sb-integration-prototype-user branch from fee9438 to 1f13966 Compare June 16, 2021 08:07
artursudnik added 15 commits June 16, 2021 11:19
@artursudnik
TODO
4cb6a39
@artursudnik
tests: implemented test for password field null value for DID users
d574b5c
@artursudnik
Merge remote-tracking branch 'origin/master' into feat/sb-integration…
d5285e0 
…-prototype-user

# Conflicts:
#	packages/ui/origin-ui-core/package.json
@artursudnik
chore: pnpm-lock.yaml updated
1d0021f
@artursudnik
chore: upgraded iam-client-lib to the 3.0.0-alpha.21 version
258a8fe
@artursudnik
chore: added user/did-roles GET endpoint
0b300fc
@artursudnik
chore: fixed iam-client-lib versions mismatch
c83cfdd
@artursudnik
chore: setting DID user role explicitly in user.controller
d436002
@artursudnik
tests: added more test scenarios, refactoring
06c9a32
@artursudnik
Merge remote-tracking branch 'origin/master' into feat/sb-integration…
adff9f9 
…-prototype-user

# Conflicts:
#	common/config/rush/pnpm-lock.yaml
@artursudnik
fix: incorrect types conversion in tests
3aaec3a
@artursudnik
docs: SB integration TODO document
9ea4775
@artursudnik
chore: added [email protected] to the origin-backend
ea3a494
@artursudnik
chore: integrated passport-did-auth login strategy into auth/login-di…
2d0961d 
…d, set DTO to match expected payload coming from the frontend
@artursudnik
tests: cleanup
f4ac1a8
@artursudnik artursudnik force-pushed the feat/sb-integration-prototype-user branch from 5579c0c to f4ac1a8 Compare June 22, 2021 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@josipbagaric josipbagaric Awaiting requested review from josipbagaric

@JonathanWfels JonathanWfels Awaiting requested review from JonathanWfels

At least 1 approving review is required to merge this pull request.

Assignees

@artursudnik artursudnik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@artursudnik @josipbagaric