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.

Sign up for GitHub

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

Jump to bottom

Logout page redirect stored in browser cache #136

Closed
ewooonk opened this issue Mar 20, 2019 · 2 comments
Closed

Logout page redirect stored in browser cache #136

ewooonk opened this issue Mar 20, 2019 · 2 comments

Comments

@ewooonk
Copy link

ewooonk commented Mar 20, 2019

The logout page 301 redirect is saved in the browser cache. Therefore the user cannot logout a second time. Redirect needs to be changed to:

header('Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0'); header("HTTP/1.1 303 See Other")
pato1 added a commit to pato1/OpenSourceBikeShare that referenced this issue Jun 12, 2019
@pato1
cyklokoalicia#136 Fix incorrect redirect type
3aefd53
@pato1 pato1 mentioned this issue Jun 12, 2019
Merged
mmmaly added a commit that referenced this issue Jun 12, 2019
@mmmaly
Merge pull request #137 from pato1/bugfix/136/logout-redirect
9d75ff6 
#136 Fix incorrect redirect type
@nekromoff
Copy link
Collaborator

302 is the correct code, not 303

@nekromoff
Copy link
Collaborator

fixed by the latest commit/pull request, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nekromoff @ewooonk