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

authenticate with JWT #8

Merged
merged 5 commits into from
Oct 30, 2023
Merged

authenticate with JWT #8

merged 5 commits into from
Oct 30, 2023

Conversation

david-eom
Copy link
Contributor

@david-eom david-eom commented Oct 28, 2023 •
edited
Loading

this pr creates login flow and adds JWT authentication to matching, question & user service
token is generated upon login and kept in local storage

steps to follow:

  1. migrate the user db (by either dropping & creating db again or adding the isAdmin column manually)
  2. create a new env file under thebackend/.env with the line JSON_WEB_TOKEN_SECRET=<secret>. all three services will be reading the env variable from this file.
  3. to create admin account, use sql to change the field for the time being

@david-eom david-eom marked this pull request as ready for review October 28, 2023 08:54
@sebbycake sebbycake mentioned this pull request Oct 30, 2023
Merged
@david-eom david-eom merged commit 60f78aa into master Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sebbycake sebbycake sebbycake approved these changes

@bnjmnt4n bnjmnt4n Awaiting requested review from bnjmnt4n

@nicole-luo-exe nicole-luo-exe Awaiting requested review from nicole-luo-exe

@fannyjian fannyjian Awaiting requested review from fannyjian

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@david-eom @sebbycake