forked from ueberauth/ueberauth_google
-
Notifications
You must be signed in to change notification settings - Fork 3
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.
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
[DLVR-6993] merge upstream #6
Open
maxp36
wants to merge
104
commits into
master
Choose a base branch
from
merge-upstream
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- for example, redeeming the code again would crash because the strategy was using OAuth2.Client.get_token!, which will attempt to illegially raise an exception using a OAuth response struct
See https://developers.google.com/identity/protocols/googlescopes for more information in the "Google Sign-In" section.
Corrected a minor typo in the README
…auth#54) * Create travis.yml with language: elixir * Updated .travis.yml Trying again, based on other elixir .travis.yml files and reading the TravisCI docs. * Converted elixir versions to strings in travis.yml
* Amended oAuth strategy to resolve m/f/a tuples - this allows specifying {m, f, a} tuples for things such as Client ID / Client Secret * Moved config resolution to after all values have been merged * Amended Strategy to also derive crednetials from options - This is in addition to deriving from the Environment
Added information about reading client_id/client_secret from MFA to the README
This reverts commit 381bc29.
…alizer Set json library on OAuth2
Add support for optional login_hint param
Prefer Local Over Global Configuration
Remove extra redirection. See erlef/setup-beam#20. Also added GHA badge to readme.
…d-bump-gha Use erlef/setup-beam and bump GHA versions
…sion-not-found-on-ci Fix "Requested Erlang/OTP version (22.3) not found in version list " on CI
…onse-with-no-error-description Handle %OAuth2.Response{status_code: 503} with no error_description in get_access_token
Add the option to get the client secret dynamically.
Add support to hl param in authorization URL
Closed
maxp36
force-pushed
the
merge-upstream
branch
from
November 6, 2023 12:21
946db01
to
88394f7
Compare
Оставлю пулик для истории, проблему решили другим способом. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Пробуем обновить модуль для гугла, чтобы порешать проблемки в это задачке https://aviasales.atlassian.net/jira/software/c/projects/DLVR/boards/377?quickFilter=1187&selectedIssue=DLVR-6993