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.

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

authorize-path and clarity #78

Open
ArsLucidum opened this issue May 14, 2015 · 2 comments
Open

authorize-path and clarity #78

ArsLucidum opened this issue May 14, 2015 · 2 comments

Comments

@ArsLucidum
Copy link

I've had some problems making the directive work with google. It took me a while to figure out that the cause was not using the authorize-path attribute in my directive, as it was defaulting to "/oauth/authorize" and appending that to my url, thus sending me to the wrong page.

For clarity, I think it would be better to either have it default to nothing at all, or making it more clear in the documentation that this might be a common issue.

@m00s
Copy link
Member

m00s commented May 23, 2015

Hi @kace91, seems like no one else has had the same issue.

IMO /oauth/authorize is widely used as authorization endpoint so I think we'll keep it as default value. Do you have any suggestion to make it more clear in the documentation?

@boldt
Copy link

boldt commented May 2, 2016

I had the same issue with a Keycloak server. Took me quite a while, to figure out this issse, e.g, that /oauth/authorize was added to the URL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants