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

Amended oAuth strategy to resolve m/f/a tuples #60

Merged
merged 3 commits into from
Mar 9, 2019
Merged

Amended oAuth strategy to resolve m/f/a tuples #60

merged 3 commits into from
Mar 9, 2019

Conversation

evadne
Copy link
Contributor

@evadne evadne commented Feb 27, 2019

  • this allows specifying {m, f, a} tuples for things such as Client ID / Client Secret

@evadne
Amended oAuth strategy to resolve m/f/a tuples
3d9f29c 
- this allows specifying {m, f, a} tuples for things such as Client ID / Client Secret
@evadne
Moved config resolution to after all values have been merged
29ad412
@evadne
Amended Strategy to also derive crednetials from options
269a440 
- This is in addition to deriving from the Environment
@doomspork doomspork requested a review from a team March 1, 2019 16:27
@doomspork doomspork merged commit 88b9ae8 into ueberauth:master Mar 9, 2019
@mspanc mspanc mentioned this pull request Mar 14, 2019
Merged
@christian-callau christian-callau mentioned this pull request Feb 28, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yordis yordis yordis approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@evadne @yordis @doomspork