Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:99899
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.128.182 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <14052ebf-efea-cb43-39e0-bdc30e493ff3@genkgo.nl>
References: <14052ebf-efea-cb43-39e0-bdc30e493ff3@genkgo.nl>
Date: Tue, 18 Jul 2017 15:57:03 +0200
Message-ID: <CADyq6sKC0o4PATjx8rShJpaC-qirjiCD5Mp7DDL6yJFkiYkjLA@mail.gmail.com>
To: "Frederik Bosch | Genkgo" <f.bosch@genkgo.nl>
Cc: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="001a114b2fbc19dda8055497e5ff"
Subject: Re: [PHP-DEV] [RFC] samesite cookie implementation
From: ocramius@gmail.com (Marco Pivetta)

--001a114b2fbc19dda8055497e5ff
Content-Type: text/plain; charset="UTF-8"

Hey Andrey,
On Mon, Jul 17, 2017 at 11:11 PM, Frederik Bosch | Genkgo <f.bosch@genkgo.nl
> wrote:

> LS,
>
> Today I finished writing the RFC for implementing same site cookies in
> PHP, https://wiki.php.net/rfc/same-site-cookie. I am happy to receive
> your remarks on the proposal, and improve when necessary.
>
> For those (only) interested in code, have a look at PR # 2613:
> https://github.com/php/php-src/pull/2613.
>
> For the record, I am just a messenger in this regard. Someone uploaded a
> patch for this feature in bug #72230: https://bugs.php.net/bug.php?i
> d=72230. I just took the opportunity to create a PR and the corresponding
> RFC. Credits for the code go to xistence at 0x90 dot nl.
>
> Hopefully, the samesite cookie flag will become a feature of the PHP
> language through this RFC!
>

The (already) infinite signature of this function just adds up some more
scenarios where BC compliance becomes a problem.

The current `setcookie` method has 7 parameters, of which 6 are optional.
This is already a mess, as any default value change introduced for either
forward-compliance or security issue compliance would result in a BC break.

This RFC suggests adding even more parameters (URGH), and increasing the
issue impact.

I had already expressed this issue in https://wiki.php.net/rfc/openssl_aead,
which basically passed

Honestly, API


Marco Pivetta

http://twitter.com/Ocramius

http://ocramius.github.com/

--001a114b2fbc19dda8055497e5ff--