diff options
author | Thomas Steur <tsteur@users.noreply.github.com> | 2020-06-10 23:17:10 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-06-10 23:17:10 +0300 |
commit | 5dd7d4fbc323cf8a506d928cb0827f4f7d656a2e (patch) | |
tree | e044ba23cd5184e57876bb0930f4cefde78b2170 /LegacyAutoloader.php | |
parent | 210e5662fbbd90c550eb086c72abc54f876d6b91 (diff) |
Fix falsely assumes session was already started (#16047)
fix https://wordpress.org/support/topic/an-error-occurred-session-already-started/#topic-12961322-replies
The code should actually not be needed as we added a while ago `session_status()` to correctly check if a session is already active or not. SID constant is not realiable.
In this case problem was this https://3v4l.org/bpUFK
Basically a session was started by some other plugin causing the SID constant to be set. The plugin also directly closed the session again so we still attempted to start the session (which is correct). But then Zend falsely assumes session is started because of the SID constant when there is actually no session.
SID is not reliable for this purpose and be better to rely on `session_status()` which we added a while backa
Diffstat (limited to 'LegacyAutoloader.php')
0 files changed, 0 insertions, 0 deletions