Skip to content

Implement Auth0 #38

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
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Implement Auth0 #38

wants to merge 4 commits into from

Conversation

JasonL24
Copy link
Contributor

@JasonL24 JasonL24 commented Mar 23, 2020 •
edited
Loading

Changes Proposed

  • Implemented Auth0 with Google and Github
  • Note: After logging in, the user is routed to '/' and directed back to the login page after logging out
  • Note: The routes do not match the issues exactly as I discussed different route names with Bryan in a Zoom meeting. I can switch them to the issue format if needed.

Current Usage

bryanwong8
bryanwong8 reviewed Mar 23, 2020
View reviewed changes
bryanwong8
bryanwong8 reviewed Mar 23, 2020
View reviewed changes
bryanwong8
bryanwong8 reviewed Mar 23, 2020
View reviewed changes
JasonL24 and others added 3 commits March 22, 2020 22:25
@JasonL24
fixed review comments
8500bff
@JasonL24
fixed structure
cca24a1
@JasonL24
Updated decorators and routes to use session (#50)
7ad829f 
* changed decorators on activities routes

* changed decorators on cards routes

* changed decorators on checkpoints routes

* changed decorators on classrooms/concepts/events routes

* changed decorators on hints/mc_choices/modules routes

* changed decorators on organizations/steps/topics routes

* changed decorators on tracks routes

* roles_required and roles_accepted decorators use session

* Changed getting identy to use session on some routes

* changing getting username to session on more routes

* finished adding session to get username

* added profile to the session index on all username

* did user session for more decorators

* finished getting username from flask session

* changing to auth0_auth in badges and mc_questions routes

* replaced auth0 keys with env vars

* imported redirect from flask

* roles now stored in profile

* Hard-coded the 'roles' as Student for now

* minor error fix

* roles fixes

* roles fixes 2.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@bryanwong8 bryanwong8 bryanwong8 left review comments

Reviewers whose approvals may not affect merge requirements

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

Assignees

@bryanwong8 bryanwong8

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JasonL24 @bryanwong8