User "Platonides" posted a comment on MediaWiki.r90193.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/90193#c18226
Commit summary:

Removed $wgProto. Previously, setting this undocumented global variable to 
anything other than the part of $wgServer before the first colon would cause 
various things to subtly screw up. Similarly, forgetting to set it when you 
override $wgServer in LocalSettings.php would break things too.

Exposing it in the default LocalSettings.php as I did in r90105 was not a good 
solution, really the only way to avoid breakage is to just get the protocol 
from $wgServer whenever you need the protocol.

Fixed $wgCookieSecure so that it will be enabled automatically if the user sets 
$wgServer to an https URL in LocalSettings.php. Added documentation for other 
cookie-related globals.

Grep indicates that $wgProto is not used by any extensions. $wgCookieSecure is 
used, hence the need for the Setup.php patch.

Comment:

$wgProto needs to be removed from extensions/Configure

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to