WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 15 months ago

#15330 closed defect (bug) (wontfix)

WP Cookies and Authenticated Access

Reported by: filosofo Owned by:
Milestone: Priority: normal
Severity: major Version: 3.1
Component: Security Keywords: has-patch close
Focuses: Cc:

Description (last modified by filosofo)

This has been an issue for a while as far as I can tell, but it's been brought to my attention by the convergence of the admin bar and Firesheep. To summarize, FORCE_SSL_ADMIN gives users a false sense of security.

What Happens

Let's say you define FORCE_SSL_ADMIN to be true. This redirects login and wp-admin access to its equivalent SSL/TLS version, mostly. Good so far.

However, when you authenticate to WP thus you get two cookies:

  • SECURE_AUTH_COOKIE (a secure cookie, to be transmitted only over a secure connection.)
  • LOGGED_IN_COOKIE (a non-secure cookie, to be transmitted over both secure and insecure connections.)

When you request a front-end page, not over SSL, your browser sends the LOGGED_IN_COOKIE but not SECURE_AUTH_COOKIE. This is sufficient to authenticate you as a user, with all the capabilities of your user (such as manage_options or edit_posts).

Why We Should Be Concerned

On an insecure network (e.g., a coffee shop), it's easy enough to get someone's LOGGED_IN_COOKIE. This allows you to do a number of things, depending on the themes and plugins used:

  • You can access user information, which ranges from the trivial (the admin bar's comment and update counts) to more significant (view draft posts).
  • You can get the logout nonce at least, and possibly more. This opens up some XSS vectors.
  • Make admin-ajax requests. All you need are the appropriate nonces, and you can perform any of the admin-ajax actions. Some of the admin-ajax actions require no nonces at all; you can e.g. query tag information.
  • The sky's the limit with themes and plugins.
    • Plugins like BuddyPress let you do practically everything on the front-end, including editing user information and handling users.
    • More generally, plugins tend to check just capabilities and verify nonces; they don't usually care about whether the user is using secure authentication. With the advent of the admin bar, we can expect even more administrative functionality to appear on the front-end, courtesy of plugins that are adding to the admin bar.

Some Suggestions

  • Simpler
    • When a site is FORCE_SSL_ADMIN, do not allow LOGGED_IN_COOKIE to be insecure. Or better, just have one authentication cookie that is secure for FORCE_SSL_ADMIN.
    • Generate different nonces depending on whether they're over SSL or not.
    • From the admin, link to the SSL versions of front-end objects, so authenticated users can browse them, secure and authenticated.
  • More Involved
    • When FORCE_SSL_ADMIN, deny authentication for capabilities above a certain level. So maybe if you have subscriber capabilities you can get a LOGGED_IN_COOKIE but not above.
    • When FORCE_SSL_ADMIN, return false for any capability check above a certain level, when is_ssl() is false. So for example, current_user_can( 'read' ) would be true, but current_user_can( 'edit_posts' ) returns true only if is_ssl().

Attachments (3)

make-logged-in-cookie-secure.15330.diff (1.6 KB) - added by filosofo 3 years ago.
15330.diff (2.8 KB) - added by ryan 3 years ago.
Allow plugins more control over secure cookies.
security.php (545 bytes) - added by ryan 3 years ago.
Example use of the filters

Download all attachments as: .zip

Change History (18)

comment:1 filosofo3 years ago

  • Description modified (diff)

comment:2 scribu3 years ago

  • Cc scribu added

comment:3 johnbillion3 years ago

  • Cc johnbillion@… added

comment:4 filosofo3 years ago

  • Keywords has-patch added

In the interests of a quick fix and the absence of any feedback, make-logged-in-cookie-secure.15330.diff makes logged-in-cookie secure when secure-auth-cookie is.

comment:5 fitztrev3 years ago

  • Cc fitztrev added

comment:6 follow-up: ryan3 years ago

I think we need to flush out FORCE_SSL_HOME and make the cookie secure if that is set.

comment:7 in reply to: ↑ 6 filosofo3 years ago

Replying to ryan:

I think we need to flush out FORCE_SSL_HOME and make the cookie secure if that is set.

I guess it depends what FORCE_SSL_HOME means. It sounds like a name for something that would serve public pages only over https.

comment:8 filosofo3 years ago

Anyways, what about my patch for the time being (3.1)? Currently it's almost impossible not to leak one's authentication credentials over an insecure connection.

comment:9 filosofo3 years ago

  • Keywords 3.2-early added
  • Milestone changed from 3.1 to Future Release

comment:10 hakre3 years ago

Related: #15928

ryan3 years ago

Allow plugins more control over secure cookies.

ryan3 years ago

Example use of the filters

comment:11 ryan3 years ago

(In [17227]) Add some cookie filters to allow plugins more control over SSL cookie delivery. see #15330

comment:12 hakre3 years ago

Related: #16869

comment:13 lancehudson2 years ago

  • Cc lance@… added

comment:14 wonderboymusic15 months ago

  • Keywords close added; 3.2-early removed
  • Milestone Future Release deleted

I'm inclined to think this is not an issue. I have implemented custom cookies and custom auth on eMusic and absolute everything can be controlled by the developer because basically everything is pluggable now.

Suggesting close since FORCE_SSL_HOME has not sprung to life in the past 2 years.

Version 0, edited 15 months ago by wonderboymusic (next)

comment:15 scribu15 months ago

  • Resolution set to wontfix
  • Status changed from new to closed

@wonderboymusic: Either

a) only add the 'close' tag

or

b) actually set the resolution to wontfix/maybelater etc. and remove the milestone

Note: See TracTickets for help on using tickets.