Making WordPress.org

Opened 5 years ago

Closed 5 years ago

#4583 closed defect (bug) (fixed)

Cookie Policy - The *.trac.wordpress.org cookie table is improperly formatted

Reported by: garrett-eclipse's profile garrett-eclipse Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: Priority: normal
Component: General Keywords: has-patch
Cc:

Description

Hello,

I noticed the *.trac.wordpress.org cookie table is slightly broken with the Purpose for trac_form_token being in the Logged-in Users Only? column.

Uploading a patch to address this.

Cheers

Attachments (3)

Screen Shot 2019-07-05 at 10.20.47 AM.png (63.5 KB) - added by garrett-eclipse 5 years ago.
Screenshot of the broken table
4583.diff (727 bytes) - added by garrett-eclipse 5 years ago.
Patch to fix the table format for *.trac.wordpress.org
4583.2.diff (1.1 KB) - added by garrett-eclipse 5 years ago.
Additional patch to fix the verbiage as it's incorrect for trac_form_token as well as add the trac_session cookie

Download all attachments as: .zip

Change History (5)

@garrett-eclipse
5 years ago

Screenshot of the broken table

@garrett-eclipse
5 years ago

Patch to fix the table format for *.trac.wordpress.org

@garrett-eclipse
5 years ago

Additional patch to fix the verbiage as it's incorrect for trac_form_token as well as add the trac_session cookie

#1 @garrett-eclipse
5 years ago

  • Keywords has-patch added

I've uploaded 4583.diff to just fix the table.

While reviewing the table though I noted that the description for trac_form_token is incorrect so I've updated it. I also noticed that trac_session is also set for the *.trac.wordpress.org sites to keep anonymous session information. I was going to also add trac_auth as it's a normal cookie for Trac but with WP Trac the wp-login is used instead. 4583.2.diff corrects these.

#2 @SergeyBiryukov
5 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 9028:

WordPress.org Main Theme: Privacy: Correct trac_form_token cookie purpose, add information about trac_session cookie.

Props garrett-eclipse.
Fixes #4583.

Note: See TracTickets for help on using tickets.