]> source.dussan.org Git - nextcloud-server.git/commit
Do not clear CSRF token on logout (fix for #1303) 3832/head
authorRoeland Jago Douma <roeland@famdouma.nl>
Mon, 13 Mar 2017 21:14:11 +0000 (22:14 +0100)
committerRoeland Jago Douma <roeland@famdouma.nl>
Mon, 13 Mar 2017 21:16:56 +0000 (22:16 +0100)
commitbb94b39745cc1b38dcc6fb12b7ec028cde3a4a11
treeda73be1424f39847e27c6d7730fd909e4c383be7
parentaf16416de51a40e3fbb1e8499bbe8bf3b77ab5af
Do not clear CSRF token on logout (fix for #1303)

This is a hacky way to allow the use case of #1303.

What happens is

1. User tries to login
2. PreLoginHook kicks in and figures out that the user need to change
their LDAP password or whatever => redirects user
3. While loading the redirect some logic of ours kicks in and logouts
the user (thus clearing the session).
4. We render the new page but now the session and the page disagree
about the CSRF token

This is kind of hacky but I don't think it introduces new attack
vectors.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
lib/private/Session/CryptoSessionData.php