#28427 closed enhancement (fixed)
All cookies should be secure when `home` and `siteurl` use HTTPS
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.0 | Priority: | low |
Severity: | minor | Version: | |
Component: | Security | Keywords: | has-patch needs-testing |
Focuses: | Cc: |
Description
In the situation where a site is only served over SSL (ie. the home
and siteurl
options use the HTTPS scheme) then all cookies should have the secure
flag set.
Currently the secure
flag isn't set on the test cookie and the settings cookies in this situation.
Somewhat related: #28426.
Attachments (3)
Change History (10)
#3
@
11 years ago
- Keywords has-patch needs-testing added
28427.diff tackles this. Note that it relies on my patch for is_https() on #28487.
The patch sets the 'secure' flag on...
- The test cookie if both
home_url()
andsite_url()
are https. - The settings cookies if
site_url()
is https. - The post password cookie if
home_url()
is https. - The comment author cookies if the comment post permalink is https.
I'm in two minds about the comment author cookies. It could just check for https on home_url()
rather than the current comment post permalink.
#5
@
11 years ago
- Owner set to johnbillion
- Resolution set to fixed
- Status changed from new to closed
In 28895:
This ticket was mentioned in IRC in #wordpress-dev by johnbillion. View the logs.
11 years ago
Note: See
TracTickets for help on using
tickets.
Also applies to the
comment_author
andcomment_author_email
cookies.