Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

SECURITY_POST_LOGIN_VIEW doesn't redirect as expected the FIRST time #263

Closed
chasrmartin opened this issue Jun 8, 2014 · 9 comments
Closed

Comments

@chasrmartin
Copy link

I'm pewrfectly willing to believe this is something I've just missed. But I have an app using Flask-Security. I have the configuration set with

app.config['SECURITY_POST_LOGIN_VIEW'] = '/dashboard'

Logging in works successfully but directly after the login, the page redirects to /

However, if I go to the login page again, it redirects me, as desired, to /dashboard.

I've been digging at this for some hours now and I'm stymied.

@mattupstate
Copy link
Collaborator

@chasrmartin This is odd, as this is tested here

@mattupstate
Copy link
Collaborator

Althought, I might be related to #261

@mattupstate
Copy link
Collaborator

@chasrmartin this will be fixed in the next release.

@chasrmartin
Copy link
Author

Thanks, Matt, do you have any idea when the next release is coming? Or
would it be worthwhile for me to pull the development version?

On Tue, Jun 10, 2014 at 8:49 AM, Matt Wright notifications@github.com
wrote:

Closed #263 #263.


Reply to this email directly or view it on GitHub
#263 (comment)
.

@mattupstate
Copy link
Collaborator

@chasrmartin I'll try and get this fix out today, thats a really annoying one.

@chasrmartin
Copy link
Author

Grazie.

On Tue, Jun 10, 2014 at 8:53 AM, Matt Wright notifications@github.com
wrote:

@chasrmartin https://github.com/chasrmartin I'll try and get this fix
out today, thats a really annoying one.


Reply to this email directly or view it on GitHub
#263 (comment)
.

@chasrmartin
Copy link
Author

... and yeah, I can dig it, I spent several hours digging at it myself. :-)

On Tue, Jun 10, 2014 at 8:54 AM, Charles R (Charlie) Martin <
chasrmartin@gmail.com> wrote:

Grazie.

On Tue, Jun 10, 2014 at 8:53 AM, Matt Wright notifications@github.com
wrote:

@chasrmartin https://github.com/chasrmartin I'll try and get this fix
out today, thats a really annoying one.


Reply to this email directly or view it on GitHub
#263 (comment)
.

@mattupstate
Copy link
Collaborator

@chasrmartin Fixes have been released.

@chasrmartin
Copy link
Author

Bingo that got it! Thanks!

On Tue, Jun 10, 2014 at 11:07 AM, Matt Wright notifications@github.com
wrote:

@chasrmartin https://github.com/chasrmartin Fixes have been released.


Reply to this email directly or view it on GitHub
#263 (comment)
.

galeo pushed a commit to galeo/flask-security-outdated that referenced this issue Mar 4, 2020
)

I can actually reproduce the bug report - unless the OP actually overrode the default form and
didn't put in a phone number.
But that's fine - simple and easy fix.

closes: pallets-eco#262
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

2 participants