Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

* (bug 13905) Blacklist Mac IE from HttpOnly cookies; it eats them sometimes

Browse source 
God I wish this browser would finish dying. :D

The particular situation was that the session cookie was getting eaten as "disabled", thus not sent back to the server so your session state never quite happened. Other cookies on submit seemed to come in intact, but without the session cookie you'd get a big fat error message, even if you set the long-term login cookie option.

Mac/IE seems to always *see* the HttpOnly cookies, but it sometimes marks them as "disabled". It seems to be incorrectly parsing the options after the path, sometimes seeing "/;" as the path instead of "/". Failure is more likely if there's no expiration option (as with the session cookie), or if there *is* a secure option set.

Anyway, just set up a user-agent blacklist $wgHttpOnlyBlacklist and copied the Mac/IE entry over. The HttpOnly setting now gets ignored for blacklist hits as well as for old PHP versions, the check being encapsulated into wfHttpOnlySafe().

Also added some logging for cookie settings, around the setcookie() and session_set_cookie_params() calls.
This commit is contained in:
Brion Vibber 2008-05-01 20:25:17 +00:00
parent 5da81d81c5
commit 6b16f44108
4 changed files with 48 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
RELEASE-NOTES
View file

@ -233,6 +233,7 @@ it from source control: http://www.mediawiki.org/wiki/Download_from_SVN
"show hidden categories" option on
* (bug 13915) Undefined variable $wltsfield in includes/SpecialWatchlist.php
* (bug 13913) Special:Whatlinkshere now has correct HTML markup
* (bug 13905) Blacklist Mac IE from HttpOnly cookies; it eats them sometimes
=== API changes in 1.13 ===

11
includes/DefaultSettings.php
View file

@ -1497,6 +1497,17 @@ $wgDisableCookieCheck = false;
*/
$wgCookieHttpOnly = version_compare("5.2", PHP_VERSION, "<");
/**
* If the requesting browser matches a regex in this blacklist, we won't
* send it cookies with HttpOnly mode, even if $wgCookieHttpOnly is on.
*/
$wgHttpOnlyBlacklist = array(
// Internet Explorer for Mac; sometimes the cookies work, sometimes
// they don't. It's difficult to predict, as combinations of path
// and expiration options affect its parsing.
'/^Mozilla\/4\.0 \(compatible; MSIE \d+\.\d+; Mac_PowerPC\)/',
);
/** A list of cookies that vary the cache (for use by extensions) */
$wgCacheVaryCookies = array();

26
includes/GlobalFunctions.php
View file

@ -2244,6 +2244,22 @@ function wfIsLocalURL( $url ) {
return Http::isLocalURL( $url );
}
function wfHttpOnlySafe() {
global $wgHttpOnlyBlacklist;
if( !version_compare("5.2", PHP_VERSION, "<") )
return false;
if( isset( $_SERVER['HTTP_USER_AGENT'] ) ) {
foreach( $wgHttpOnlyBlacklist as $regex ) {
if( preg_match( $regex, $_SERVER['HTTP_USER_AGENT'] ) ) {
return false;
}
}
}
return true;
}
/**
* Initialise php session
*/
@ -2256,7 +2272,15 @@ function wfSetupSession() {
# application, it will end up failing. Try to recover.
ini_set ( 'session.save_handler', 'files' );
}
$httpOnlySafe = version_compare("5.2", PHP_VERSION, "<");
$httpOnlySafe = wfHttpOnlySafe();
wfDebugLog( 'cookie',
'session_set_cookie_params: "' . implode( '", "',
array(
0,
$wgCookiePath,
$wgCookieDomain,
$wgCookieSecure,
$httpOnlySafe && $wgCookieHttpOnly ) ) . '"' );
if( $httpOnlySafe && $wgCookieHttpOnly ) {
session_set_cookie_params( 0, $wgCookiePath, $wgCookieDomain, $wgCookieSecure, $wgCookieHttpOnly );
} else {

13
includes/User.php
View file

@ -2006,8 +2006,17 @@ class User {
if( $exp == 0 ) {
$exp = time() + $wgCookieExpiration;
}
$httpOnlySafe = version_compare("5.2", PHP_VERSION, "<");
$httpOnlySafe = wfHttpOnlySafe();
wfDebugLog( 'cookie',
'setcookie: "' . implode( '", "',
array(
$wgCookiePrefix . $name,
$value,
$exp,
'/',
$wgCookieDomain,
$wgCookieSecure,
$httpOnlySafe && $wgCookieHttpOnly ) ) . '"' );
if( $httpOnlySafe && isset( $wgCookieHttpOnly ) ) {
setcookie( $wgCookiePrefix . $name,
$value,