Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:118009
Message-ID: <5758628b-197f-000c-bfbf-8b6884b11223@gmx.de>
Date: Mon, 20 Jun 2022 15:44:57 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.10.0
Content-Language: de-DE
To: Pierrick Charron <pierrick@php.net>,
 Rowan Tommins <rowan.collins@gmail.com>
Cc: internals@lists.php.net
References: <CAOfKkdRzoo7JuD07F8nkZBYvS5eDMiV2r6cNV-FgeLxBrdCsiQ@mail.gmail.com>
 <CAESVnVpzqT0xtEShqQp+FPXt2DQjeAXpkSVZTACAC0CW=8FBBg@mail.gmail.com>
 <CAOfKkdTu1f6jJ2HwGjOnBR2sO6P6qdOrjH1ZzumwLzf1syXa6Q@mail.gmail.com>
 <76FA5161-16C3-4570-9EF8-B75DF2E6530A@gmail.com>
 <CAOfKkdQGX5Lf5-jKyKnh5im9YYd-pKPx=AxnLgk9=tBoJ37E9Q@mail.gmail.com>
In-Reply-To: <CAOfKkdQGX5Lf5-jKyKnh5im9YYd-pKPx=AxnLgk9=tBoJ37E9Q@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] Discussion about new Curl URL API and ext/curl
 improvements
From: cmbecker69@gmx.de ("Christoph M. Becker")

On 20.06.2022 at 15:13, Pierrick Charron wrote:

> Thanks again for your comments. PHP8.2 is going to be feature freeze in
> exactly one month now and as the currently proposed improvements have no=
t
> received any interest (and that's OK), I will put those aside for the
> moment and come back with a new approach that I hope will be better for =
the
> future release (8.3 ?)
>
> However, about the new Curl URL API, I think it's still time to finalize
> the discussions and include it in the 8.2 release as it allows us to sol=
ve
> some potential security issues.
>
> What do you think ? Do you see any interest in adding the new CurlUrl cl=
ass
> in PHP8.2 ? Again currently proposed implementation can be found here [1=
]
> and is waiting for your new comments.
>
> Pierrick
>
> [1]
> https://github.com/adoy/php-src/pull/1/files#diff-5bd329a70f563b5b6eded3=
b710514a4fac605329087a7bd118e40f80b9fc8ee3

Given that designing a new OO API obviously isn't uncontroversial, and
that there are only four weeks left till feature freeze, it might be
best to go with a simple procedural API for PHP 8.2 only.  This can be
marked as being experimental, and can still be deprecated in one of the
next minors.  That's not nice for early adopters, but still seems better
than to not introduce the CURL URL API in PHP 8.2, and certainly is
better than to rush some OO API.

Christoph