Force SessionHandler::$isACP to be false
authorTim Düsterhus <duesterhus@woltlab.com>
Fri, 8 Jan 2021 14:35:28 +0000 (15:35 +0100)
committerTim Düsterhus <duesterhus@woltlab.com>
Thu, 21 Jan 2021 10:10:02 +0000 (11:10 +0100)
commit57331187f91b7a5c77b9e8f3be929b81ec62fe5e
treef6e199a1b22e143f2e835a38f7e2c892b314c79c
parent1d56234d204a1c6a31390b2d705015ed8c0921e5
Force SessionHandler::$isACP to be false

This causes the ACP to reuse the frontend session. This improves the user
experience for enabled multi-factor authentication, because the ACP will no
longer require both the password *and* an additional MFA code when the user's
web browser is already authenticated in the frontend.

Additionally it will allow to simplify the whole session handling logic, due to
the future removal of several code branches.

This removal of the branches is not yet done to keep this commit simple.

As of right now the ACP will have reduced security compared to the situation in
5.3, because no passwords will be asked either. This will also be fixed in a
future commit by using the reauthentication framework.
com.woltlab.wcf/package.xml
wcfsetup/install/files/acp/update_com.woltlab.wcf_5.4_session_1_cookies.php
wcfsetup/install/files/lib/system/session/SessionHandler.class.php