WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#5076 closed defect (bug) (fixed)

Cookie testing prevents login

Reported by: ryan Owned by:
Milestone: 2.3.1 Priority: normal
Severity: normal Version: 2.3
Component: General Keywords: login cookie
Focuses: Cc:

Description (last modified by ryan)

http://wordpress.org/support/topic/135598?replies=11

Some are having login problems that are fixed when the cookie testing code is removed. See [6009].

Let's gather some information. Browser version, http server version, firefox extensions, anti-virus/firewall software, etc.

Attachments (1)

sitecookie.diff (535 bytes) - added by ryan 7 years ago.
Set TEST_COOKIE for SITECOOKIEPATH

Download all attachments as: .zip

Change History (9)

comment:1 ryan7 years ago

  • Description modified (diff)

comment:2 westi7 years ago

Some notes:

Original reporter on wp-testers has 3 blogs on same domain on same host (2 work - 1 doesn't)

There is a limit on the number of cookies per domain based on RFC2109:

6.3 Implementation Limits

Practical user agent implementations have limits on the number and
size of cookies that they can store. In general, user agents' cookie
support should have no fixed limits. They should strive to store as
many frequently-used cookies as possible. Furthermore, general-use
user agents should provide each of the following minimum capabilities
individually, although not necessarily simultaneously:

  • at least 300 cookies
  • at least 4096 bytes per cookie (as measured by the size of the characters that comprise the cookie non-terminal in the syntax description of the Set-Cookie header)
  • at least 20 cookies per unique host or domain name

User agents created for specific purposes or for limited-capacity
devices should provide at least 20 cookies of 4096 bytes, to ensure
that the user can interact with a session-based origin server.

The information in a Set-Cookie response header must be retained in
its entirety. If for some reason there is inadequate space to store
the cookie, it must be discarded, not truncated.

Applications should use as few and as small cookies as possible, and
they should cope gracefully with the loss of a cookie.

comment:3 westi7 years ago

Note also - setcookie will fail (possibly silently) if the headers are already sent according to http://uk.php.net/setcookie

ryan7 years ago

Set TEST_COOKIE for SITECOOKIEPATH

comment:4 follow-up: ryan7 years ago

Maybe we need to set the cookie with SITECOOKIEPATH in addition to COOKIEPATH.

comment:5 ryan7 years ago

(In [6167]) Set TEST_COOKIE for SITECOOKIEPATH. see #5076

comment:6 ryan7 years ago

(In [6168]) Set TEST_COOKIE for SITECOOKIEPATH. see #5076

comment:7 in reply to: ↑ 4 westi7 years ago

Replying to ryan:

Maybe we need to set the cookie with SITECOOKIEPATH in addition to COOKIEPATH.

I guess maybe I should have a test install in a subdirectory and maybe I would have caught this one then :-(

comment:8 ryan7 years ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.