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

URI fragment should appear at the end of URL #214

Merged

Conversation

edymerchk
Copy link
Contributor

Hi, thanks for this project !

as I mention in #213 the params in the query string for the redirect_url are not sent to the server due to the URI fragment.

according to the RFC fragment should be at the end of the URL

more info: http://en.wikipedia.org/wiki/Fragment_identifier#Examples

not sure how Angular handle this, but I found: https://scotch.io/quick-tips/pretty-urls-in-angularjs-removing-the-hashtag

@booleanbetrayal
Copy link
Collaborator

This seems correct, but there was a comment to the effect that this was required in AngularJS for some reason. Any idea what that's all about @lynndylanhurley ? I can run it through some testing in a few days if it's hazy.

@booleanbetrayal
Copy link
Collaborator

@edymerchk - Can you go ahead and rebase against master, resolving conflicts please? Thanks!

@booleanbetrayal
Copy link
Collaborator

@edymerchk - reminder to rebase against master if you'd like credit for this PR! thanks!

@edymerchk edymerchk force-pushed the fix_fragment_and_params branch 2 times, most recently from 3ac7dc9 to e7c6410 Compare July 14, 2015 13:18
@edymerchk edymerchk force-pushed the fix_fragment_and_params branch from e7c6410 to b60c7c3 Compare July 14, 2015 13:22
@edymerchk
Copy link
Contributor Author

@booleanbetrayal ready

booleanbetrayal added a commit that referenced this pull request Jul 14, 2015
URI fragment should appear at the end of URL
@booleanbetrayal booleanbetrayal merged commit 54c7890 into lynndylanhurley:master Jul 14, 2015
@booleanbetrayal
Copy link
Collaborator

Thanks @edymerchk !

@dcunited001
Copy link

+1

@dcunited001
Copy link

has this been merged into a version i can pull? i don't see it in 0.32.beta9. looks like i can bundle it with 0.32.beta10, but i don't see a tag for that one yet.

i'm pulling it down with that one now. this URL issue doesn't break the confirmation process, but it prevents the ng-token-auth event auth:email-confirmation-success from firing in android because the URL params can't be parsed.

@booleanbetrayal
Copy link
Collaborator

I'm going to be stamping out a new stable build version shortly!

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

Successfully merging this pull request may close these issues.

3 participants