WordPress.org

Make WordPress Core

Opened 5 weeks ago

Last modified 5 weeks ago

#44175 new defect (bug)

Privacy Policy Guide Text Incorrect Regarding Setting of Test Cookie

Reported by: mechter Owned by:
Milestone: 4.9.8 Priority: normal
Severity: normal Version: 4.9.6
Component: Privacy Keywords: has-patch
Focuses: Cc:

Description

The new privacy policy guide text states: If you have an account and you log in to this site, we will set a temporary cookie to determine if your browser accepts cookies.

This is incorrect. The test cookie is set regardless of whether a site visitor has an account or not and it is not set upon logging in, but on visiting wp-login.php with or without intention to log in.

Suggested fix: If you visit our login page, we will set a temporary cookie to determine if your browser accepts cookies.

Attachments (1)

44175.diff (1.9 KB) - added by subrataemfluence 5 weeks ago.

Download all attachments as: .zip

Change History (4)

#1 @SergeyBiryukov
5 weeks ago

  • Milestone changed from Awaiting Review to 4.9.7

I wonder if we could change the behavior to match the text, so that the test cookie is only set upon trying to log in, as that probably makes more sense. I'd be more comfortable with doing that in a major release though.

For 4.9.7, let's change the text to match the current behaviour.

#2 @subrataemfluence
5 weeks ago

  • Keywords has-patch added

I agree with @SergeyBiryukov that the cookie should be set on login. This is not very feasible to set it on visiting login page irrespective of the further action taken by the user. As of now, as suggested, here is the patch to change the text to match the present text.

Will work on the login script so that the cookie is set after login is successfully done.

#3 @desrosj
5 weeks ago

  • Milestone changed from 4.9.7 to 4.9.8

Moving all tickets in 4.9.7 to 4.9.8.

Note: See TracTickets for help on using tickets.