Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

[Security] Session fixation and CSRF #279

Open
zxti opened this issue Apr 12, 2020 · 0 comments
Open

[Security] Session fixation and CSRF #279

zxti opened this issue Apr 12, 2020 · 0 comments

Comments

@zxti
Copy link

zxti commented Apr 12, 2020

It is important to reset the CSRF token when authenticating as a different user, see for instance to https://security.stackexchange.com/a/22936/17247. I may be missing something but it does not appear that this starter app resets the token accordingly.

Same thing with the session itself in general, on logout there is no resetting of the session. (I don't believe that passport's .logout() method does this for you.)

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

No branches or pull requests

1 participant