jenkins-bot has submitted this change and it was merged.

Change subject: Use a UNIX timestamp in the LoggedOut cookie
......................................................................


Use a UNIX timestamp in the LoggedOut cookie

Since it is easier to interpret in VCL. The read side code strangely
does not need to be updated, and MW will continue to work with
cookies sent before this change, because wfTimestamp() automatically
detects the timestamp format.

Change-Id: I8c661b735b1de8e295d5d713a5ca959ee99e274e
---
M includes/User.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Tim Starling: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/User.php b/includes/User.php
index 3ee32c3..2e22312 100644
--- a/includes/User.php
+++ b/includes/User.php
@@ -3146,7 +3146,7 @@
                $this->clearCookie( 'forceHTTPS' );
 
                // Remember when user logged out, to prevent seeing cached pages
-               $this->setCookie( 'LoggedOut', wfTimestampNow(), time() + 86400 
);
+               $this->setCookie( 'LoggedOut', time(), time() + 86400 );
        }
 
        /**

-- 
To view, visit https://gerrit.wikimedia.org/r/65593
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8c661b735b1de8e295d5d713a5ca959ee99e274e
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Tim Starling <tstarl...@wikimedia.org>
Gerrit-Reviewer: Parent5446 <tylerro...@gmail.com>
Gerrit-Reviewer: Tim Starling <tstarl...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to