-
Notifications
You must be signed in to change notification settings - Fork 5
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
callback url does not work proberly #3
Comments
It would be nice to have a fix for this issue :D (please) |
Took me a while to figure this out and reach this page. This is an incompatibility with the latest omniauth-oauth2 gem. Downgrading omniauth-oauth2 to 1.3.0 fixes this issue. The issue is discussed here: omniauth/omniauth-oauth2@2615267 and mentions possible workarounds. |
* Fix compatibility with omniauth-oauth 1.4+ References: * omniauth/omniauth-oauth2#70 * omniauth/omniauth-oauth2@2615267 * omniauth/omniauth-oauth2#82 * jdennes/omniauth-createsend#3 * WebTheoryLLC/omniauth-twitch#4 * jdennes/omniauth-createsend#5 * DripEmail/omniauth-drip#6 * Don't force https if it's localhost * Allow 127.0.0.1 and localhost to be on http * Lookup IP for the given host and force ssl Avoid to force ssl for local IPs (127/8 network) * Improve resolving * Improve even more * Fix regexp * Allow fe80::* entirely
Hi, was this ever fixed? I'm still getting the same problem. |
@ManuelFrigerio yes it is fixed but not merged here: #5 |
Hi.
When i use this strategy, i get the following error after i login.
Happen when it tries to get a token.
When i check what redirect uri i sent i get:
http://seetest.dev:3006/auth/createsend/callback?code=a_code&state=a_state
If i set manualy the redirect uri in
gems/oauth2-1.2.0/lib/oauth2/client.rb
to :http://seetest.dev:3006/auth/createsend/callback
the flow to get the token will workIs that a problem with this strategy or with the api wich doesn't allow redirect url with params?
Thanks in advance
The text was updated successfully, but these errors were encountered: