Skip to content

Support setting whitelist, without setting default redirect_url #894

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

Merged

Conversation

dkniffin
Copy link
Contributor

@dkniffin dkniffin commented May 23, 2017

Error:

Redirect to '' not allowed

Steps to reproduce

  1. set redirect_whitelist (to ['google.com'], for example)
  2. don't set default_confirm_success_url
  3. Try and create a new user via POST /

Expected outcome

A successful registration

Actual outcome

You should get Redirect to '' not allowed

This change resolves the issue.

@zachfeldman
Copy link
Contributor

Easy to review this one!

@MaicolBen MaicolBen merged commit 1f48c0c into lynndylanhurley:master Oct 9, 2017
@zachfeldman
Copy link
Contributor

👍

@dkniffin dkniffin deleted the nil-default-redirect-url branch October 11, 2017 12:20
@dkniffin
Copy link
Contributor Author

Thanks, guys!

@dkniffin dkniffin restored the nil-default-redirect-url branch October 11, 2017 14:16
@dkniffin
Copy link
Contributor Author

@zachfeldman Any chance we can get a version bump soon for this?

@MaicolBen
Copy link
Collaborator

@dkniffin We're testing the last code in master in order to release a new version soon

@zachfeldman
Copy link
Contributor

Coming soon #972 @lynndylanhurley has the keys to Rubygems :)

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

Successfully merging this pull request may close these issues.

3 participants