Make WordPress Core

Opened 4 years ago

Closed 2 years ago

#55440 closed enhancement (duplicate)

Introduce a wrapper fro setcookie()

Reported by: Chouby Owned by:
Priority: normal Milestone:
Component: General Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Recent browsers expect the parameter samesite to be set. A new parameter sameparty is on its way too.

The old (PHP < 7.3) setcookie() signature doesn't accept these new parameters. It's however possible to set them with a hack.

I propose to introduce a new wp_set_cookie() function to support all versions of PHP. I also propose to introduce a filter in this function. This should help for #37000.

Attachments (1)

55440.patch (2.4 KB ) - added by Chouby 4 years ago.

Download all attachments as: .zip

Change History (4)

@Chouby
4 years ago

#1 @Chouby
4 years ago

  • Keywords has-patch added

The first patch is proposed for feedback on the function itself. It doesn't include the replacement of the current calls to setcookie() by wp_set_cookie().

This ticket was mentioned in Slack in #core by sebastienserre. View the logs.


2 years ago

#3 @swissspidy
2 years ago

  • Milestone Awaiting Review
  • Resolutionduplicate
  • Status newclosed

The proposed patch is pretty similar to the ones from #37000, and ultimately both tickets are trying to address the same issue.

Note: See TracTickets for help on using tickets.