Skip to content
This repository has been archived by the owner on Sep 12, 2021. It is now read-only.

Some OAuth2 based improvements #528

Merged
merged 1 commit into from
Jul 30, 2017
Merged

Some OAuth2 based improvements #528

merged 1 commit into from
Jul 30, 2017

Conversation

akkie
Copy link
Contributor

@akkie akkie commented Jul 30, 2017

Fixes

Fixes #527

Purpose

  • Show provider response(body, status) in exception message for unexpected responses
  • Remove Clef provider
  • Do not validate social state if no state handler is registered

References

Are there any relevant issues / PRs / mailing lists discussions?

http://discourse.silhouette.rocks/t/silhouette-and-angular/159/12

@akkie
Some OAuth2 based improvements
2ed0070 
- Show provider response(body, status) in exception message for unexpected responses
- Remove Clef provider
- Do not validate social state if no state handler is registered
@coveralls
Copy link

coveralls commented Jul 30, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.04%) to 95.493% when pulling 2ed0070 on akkie:oauth2-improvements into ca9d1ff on mohiva:master.

@akkie akkie merged commit cfa90d8 into mohiva:master Jul 30, 2017
@akkie akkie deleted the oauth2-improvements branch July 30, 2017 12:03
@akkie akkie added this to the 5.0.0 milestone Jul 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
5.0.0
2 participants
@akkie @coveralls