Make WordPress Core

Opened 9 years ago

Closed 9 months ago

Last modified 9 months ago

#38711 closed defect (bug) (wontfix)

Invalid Cookie domain for wp-settings-time-*

Reported by: sebastianpisula's profile sebastian.pisula Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Users Keywords: has-patch needs-testing reporter-feedback
Focuses: administration Cc:

Description

I have set cookie domain

<?php
define( 'COOKIE_DOMAIN', '.example.com' );

but wp-settings-time-* have invalid cookie domain.

Attachments (1)

38711.patch (1012 bytes) - added by sebastian.pisula 9 years ago.

Download all attachments as: .zip

Change History (7)

#1 @johnbillion
9 years ago

  • Component changed from General to Users
  • Focuses administration added
  • Keywords has-patch needs-testing reporter-feedback added

Thanks for the patch.

What does the cookie domain get set to without this patch?

#2 @sebastian.pisula
9 years ago

I have defined in wp-config.php

<?php
define( 'COOKIE_DOMAIN', '.example.com' );

my address: http://crm.example.com

Cookie domain should be .example.com but is crm.example.com

This patch fix this.

This ticket was mentioned in PR #9267 on WordPress/wordpress-develop by @sainathpoojary.


9 months ago
#3

Trac ticket: #38711

#4 @sainathpoojary
9 months ago

Tested and confirmed the issue still exists. The original patch didn’t apply cleanly, refreshed it and raised a PR for review.

#5 @siliconforks
9 months ago

It appears that the current behavior is actually intentional: see [29478].

#6 @johnbillion
9 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Good spot. Closing as per [29478] / #29095.

Last edited 9 months ago by sabernhardt (previous) (diff)
Note: See TracTickets for help on using tickets.