-
-
Notifications
You must be signed in to change notification settings - Fork 208
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
LoginFilter messes with get's token parameter (reset-password) #94
Comments
Could you provide your filter setup? I'll have to dig into it but would like that additional info to make sure we're on the same page. |
\App\Config\Filters
\App\Config\Routes
And finally, \Myth\Auth\Filters\LoginFilter
|
It seems the problem had to do with a commented line in |
Scenario:
App\Config\Filters
)before($request)
method avoids checking for logged in user when route islogin
,forgot
orreset-password
So far, everything works as expected. But, when I click on the reset form link (
http://whatever.com/reset-password?token=e6290a3d8d156339963092e08228f039
) I keep being redirected to login form.Routes work as expected too. If, for instance, I write the URL like
http://whatever.com/reset-password
I get the reset-password form OK.If I write the URL like
http://whatever.com/reset-password?
works fine too.And so on...
...until I write
http://whatever.com/reset-password?token=
Then, it redirects to login form. Whatever I write after the = sign makes no difference
Now, if I log-in, I am redirected to the reset-password form and the token input field gets populated just fine.
Tried
d($request)
first thing inside thebefore()
method of LoginFilter and URL is fine.But when I
d($request)
after thecurrent_url()
check and before thereturn
, d($request) shows /login as the URLAlso, if I get rid of the global
login
filter, everything works as expected.Any ideas?
The text was updated successfully, but these errors were encountered: