Skip to content
This repository was archived by the owner on Oct 8, 2022. It is now read-only.

feat: auth #26

Open
moreirathomas wants to merge 20 commits into
base: main
Choose a base branch
from
Open

feat: auth #26

moreirathomas wants to merge 20 commits into from

Conversation

@moreirathomas
Copy link
Member

@moreirathomas moreirathomas commented Mar 11, 2022 •
edited
Loading

Description

🚧

  • OAuth stuff
    • exchange code for access_token from GitHub oauth
    • get user data from GitHub API (name and email for now)
    • test it
    • use actual client_id and client_secret
  • jwt stuff
    • build claims
    • sign token
    • verify token
    • extract claims from token
    • test it
  • routes stuff
    • connect handlers
    • write auth middleware
    • protect authenticated routes
  • data service stuff
    • define services
    • use services in handlers
    • implement services in db facade

Changes

Notes

Closes #4 and relates to #5, although db stuff isn't in this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@c1-ra c1-ra Awaiting requested review from c1-ra

@GregoryAlbouy GregoryAlbouy Awaiting requested review from GregoryAlbouy

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

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Authenticate the user when he posts a test and when he get a test visualization

2 participants

@moreirathomas