Remove cleanup of `userID` and `password` cookies
authorTim Düsterhus <duesterhus@woltlab.com>
Mon, 5 Jul 2021 14:00:08 +0000 (16:00 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Mon, 5 Jul 2021 14:00:08 +0000 (16:00 +0200)
They are no longer used as of WoltLab Suite 5.4, however the removal remained
in the code to clean up the browser's cookie store on logout. Remove it, the
cookies will naturally expire in the future.

wcfsetup/install/files/lib/action/LogoutAction.class.php
wcfsetup/install/files/lib/system/WCF.class.php

index 77d566ec732a8789ff5b749500a5c4bce7c9bf12..690dc9f8766d83abaa7f5b665eda4ea78cd01bdb 100644 (file)
@@ -27,14 +27,6 @@ class LogoutAction extends \wcf\acp\action\LogoutAction
 
         WCF::getSession()->delete();
 
-        // remove cookies
-        if (isset($_COOKIE[COOKIE_PREFIX . 'userID'])) {
-            HeaderUtil::setCookie('userID', 0);
-        }
-        if (isset($_COOKIE[COOKIE_PREFIX . 'password'])) {
-            HeaderUtil::setCookie('password', '');
-        }
-
         $this->executed();
 
         // forward to index page
index 209b129543f75b087cc48a1438d2fd3482343d38..de18f14cd17fac6b652085d015220ba07d5617ff 100644 (file)
@@ -554,14 +554,6 @@ class WCF
             } else {
                 self::$forceLogout = true;
 
-                // remove cookies
-                if (isset($_COOKIE[COOKIE_PREFIX . 'userID'])) {
-                    HeaderUtil::setCookie('userID', 0);
-                }
-                if (isset($_COOKIE[COOKIE_PREFIX . 'password'])) {
-                    HeaderUtil::setCookie('password', '');
-                }
-
                 throw new NamedUserException(self::getLanguage()->getDynamicVariable('wcf.user.error.isBanned'));
             }
         }