/oauth2/v1/userinfo URL is hard-coded

omniauth / omniauth-okta

OAuth2 strategy for Okta

MIT License

41 stars 37 forks source link

/oauth2/v1/userinfo URL is hard-coded #6

Closed matthewkrom closed 2 years ago

matthewkrom commented 5 years ago

Hello,

I notice /oauth2/v1/userinfo in #raw_info is a hard-coded URL. I have a case where it needs to include the auth server ID e.g. /oauth2//v1/userinfo. authorize_url and token_url are both configurable through client_options. Would it make sense to add userinfo_url to this list? I could submit a pull request, but first wanted to ask for feedback, or if I'm missing something.

dandrews commented 5 years ago

That's a fair point. Please submit a pull request that abstracts the URL/domain into a variable/config var.