Context Navigation

← Previous Change
Next Change →

option.php

Timestamp:

06/29/2014 01:24:55 PM (12 years ago)

Author:

johnbillion

Message:

Conditionally set the the secure flag on the test cookie, post password cookie, settings cookies, and comment author cookies depending on whether the front end and/or admin area are served over https. Fixes #28427

File:

: 1 edited

trunk/src/wp-includes/option.php (modified) (1 diff)

Legend:

: Unmodified
: Added
: Removed

trunk/src/wp-includes/option.php

-              r28735
+              r28895
         // The cookie is not set in the current browser or the saved value is newer.
+        setcookie( 'wp-settings-' . $user_id, $settings, time() + YEAR_IN_SECONDS, SITECOOKIEPATH );
+        setcookie( 'wp-settings-time-' . $user_id, time(), time() + YEAR_IN_SECONDS, SITECOOKIEPATH );
+        $secure = is_https_url( site_url() );
+        setcookie( 'wp-settings-' . $user_id, $settings, time() + YEAR_IN_SECONDS, SITECOOKIEPATH, COOKIE_DOMAIN, $secure );
+        setcookie( 'wp-settings-time-' . $user_id, time(), time() + YEAR_IN_SECONDS, SITECOOKIEPATH, COOKIE_DOMAIN, $secure );
         $_COOKIE['wp-settings-' . $user_id] = $settings;
+}

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 28895 for trunk/src/wp-includes/option.php

Legend:

trunk/src/wp-includes/option.php

Download in other formats: