Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Issue-60: postResetUrl not working #78

Merged
merged 1 commit into from
Oct 3, 2017
Merged

Issue-60: postResetUrl not working #78

merged 1 commit into from
Oct 3, 2017

Conversation

ddelponte
Copy link
Collaborator

@ddelponte ddelponte commented Oct 2, 2017

Documentation suggests to use grails.plugin.springsecurity.ui.forgotPassword.postResetUrl in order to set the URL after a password reset. For example:

grails.plugin.springsecurity.ui.forgotPassword.postResetUrl = '/reset'

In RegisterController the registerPostResetUrl variable looks for conf.ui.register.postResetUrl which should be conf.ui.forgotPassword.postResetUrl.

https://github.com/grails-plugins/grails-spring-security-ui/blob/master/grails-app/controllers/grails/plugin/springsecurity/ui/RegisterController.groovy#L212

Updated RegisterController.registerPostResetUrl to be:

registerPostResetUrl = conf.ui.forgotPassword.postResetUrl ?: ''

Documentation suggests to use grails.plugin.springsecurity.ui.forgotPassword.postResetUrl in order to set the URL after a password reset. For example:

----
grails.plugin.springsecurity.ui.forgotPassword.postResetUrl = '/reset'
----
In RegisterController the registerPostResetUrl variable looks for conf.ui.register.postResetUrl which should be conf.ui.forgotPassword.postResetUrl.

https://github.com/grails-plugins/grails-spring-security-ui/blob/master/grails-app/controllers/grails/plugin/springsecurity/ui/RegisterController.groovy#L212

Updated `RegisterController.registerPostResetUrl` to be:
```
registerPostResetUrl = conf.ui.forgotPassword.postResetUrl ?: ''
```
@ddelponte ddelponte mentioned this pull request Oct 2, 2017
@ddelponte ddelponte requested a review from sdelamo October 2, 2017 19:24
@sdelamo sdelamo merged commit b6d6d88 into master Oct 3, 2017
@sdelamo sdelamo deleted the issue-60 branch October 3, 2017 11:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants