From cecc08756e61af4401f459c99038ceb563d5180e Mon Sep 17 00:00:00 2001 From: epriestley Date: Thu, 8 Jan 2015 16:24:34 -0800 Subject: [PATCH] Fix an issue with clearing cookies when a cookie prefix is set Summary: If a cookie prefix is set (as on the Phacility cluster), we end up double-namespacing cookies when trying to remove them. This can make logging out produce a cookie error. Test Plan: Logged out locally with cookie prefix, got normal logout workflow. Reviewers: btrahan Reviewed By: btrahan Subscribers: epriestley Differential Revision: https://secure.phabricator.com/D11282 --- src/aphront/AphrontRequest.php | 1 - 1 file changed, 1 deletion(-) diff --git a/src/aphront/AphrontRequest.php b/src/aphront/AphrontRequest.php index a9800a66f2..6e167c64e5 100644 --- a/src/aphront/AphrontRequest.php +++ b/src/aphront/AphrontRequest.php @@ -333,7 +333,6 @@ final class AphrontRequest { } final public function clearCookie($name) { - $name = $this->getPrefixedCookieName($name); $this->setCookieWithExpiration($name, '', time() - (60 * 60 * 24 * 30)); unset($_COOKIE[$name]); }