Closed sabondano closed 5 years ago
Thank you for contributing!
I'll have to take a look later this week if you don't mind.
@sntran My pleasure 😊. No problem and thank you!
Thank you for contributing!
I'll have to take a look later this week if you don't mind.
👋 @sntran did you have a chance to look at this last week 😅?
Thanks again @sabondano !
Why:
Now that Ueberauth version 0.6+ supports per-app configuration, we'd like to update uberauth_auth0 strategy to support per-app configuration also.
We'd like to config our apps like this, in our configuration file (config/config.exs):
and to use
Ueberauth
from a controller like this:Issue link: n/a
This change addresses the need by:
Ueberauth.Strategy.Auth0.handle_request!/1
andUeberauth.Strategy.Auth0.handle_callback!/1
to pass along the configured:otp_app
, found in theconn
, to a function that needs it in the body of these two functions.Ueberauth.Strategy.Auth0.OAuth
to get config fromotp_app
when anotp_app
is provided toauthorize_url!/2
andget_token!/2
.