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

Rework strategy to handle OAuth and OIDC #44

Merged
merged 12 commits into from
Jan 25, 2017
Merged

Rework strategy to handle OAuth and OIDC #44

merged 12 commits into from
Jan 25, 2017

Conversation

hzalaz
Copy link
Member

@hzalaz hzalaz commented Jan 25, 2017

  • Reworked how it's built, it nows require client_id, client_secret and auth0 domain
  • Handles absence of id_token
  • By default sets no scope or audience
  • Parses /userninfo response and handles non-oidc payload

@hzalaz
Cleaned up project structure.
e84bdf2 
- Updated gems
- gemspec only has direct dependencies of the gem.
- Dev and Test dependencies are in Gemfile
- Removed tests (will write from scratch)
@hzalaz
Setup Guard & remove old examples
ca60194
@hzalaz
Set correct version for travis
a674c58
@hzalaz
Add sinatra app as example
00b9552
@hzalaz
Basic Auth0 strategy based on OAuth 2.0
0e8721e 
- Require client_id, client_secret and domain on init
- Use auth0 paths for OAuth
- Handled non-http domain
- Guardfile and test
@hzalaz
Validate arguments of strategy
129ada1
@hzalaz
Fix rubocop errors
73ebb1f
@hzalaz
Request user information from /userinfo
306bcb4 
Parse both basic response (sub only) and OIDC
@hzalaz
Parse credentials properly from Auth0 reponse
c14e40d 
Return all tokens (id, access and refresh) if available
@hzalaz
Clean up files and styles
1e1df07
@hzalaz
Add telemetry
e406da3
@hzalaz
Restrict only master branch for CI pushes
cd9bf5e
@hzalaz hzalaz merged commit a3abcc5 into master Jan 25, 2017
@hzalaz hzalaz deleted the prepare--for-oidc-version branch January 25, 2017 14:57
@dependabot-preview dependabot-preview bot mentioned this pull request Mar 21, 2018
Closed
@joshcanhelp joshcanhelp added this to the 2.0.0 milestone Oct 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CH: Fixed
Projects
None yet
Milestone
2.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@hzalaz @joshcanhelp