Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:118581
MIME-Version: 1.0
References: <CABiw4UhL6NTD+iPPY=9qwN5-bKABkLxgrPDA2awO171Txi1-eQ@mail.gmail.com>
 <60350e90-3f2a-df6b-fd56-ab6613a610eb@gmx.de>
In-Reply-To: <60350e90-3f2a-df6b-fd56-ab6613a610eb@gmx.de>
Date: Thu, 8 Sep 2022 11:35:44 +0100
Message-ID: <CAEKnhAG4zotAKwa++nTdH9r3QOi95_6Dw57bCKN-hRo_AXc45w@mail.gmail.com>
To: "Christoph M. Becker" <cmbecker69@gmx.de>
Cc: Misha <mishanon@gmail.com>, PHP internals list <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000008f135605e827fe47"
Subject: Re: [PHP-DEV] Re: Increase maximum size of an uploaded file to 50Mbyte
From: bukka@php.net (Jakub Zelenka)

--0000000000008f135605e827fe47
Content-Type: text/plain; charset="UTF-8"

On Wed, Sep 7, 2022 at 3:28 PM Christoph M. Becker <cmbecker69@gmx.de>
wrote:

> On 07.09.2022 at 15:57, Misha wrote:
>
> > We spend a lot of time to increase limits for uploads file in PHP. Can we
> > increase it in php.ini?
> >
> > Current value is 2Mb. Its so small value, when photo image can take 8Mb
> on
> > iPhone X.
> > We should increase it to 50Mb, because DevOps engineers do useless work
> > trying to change it.
> >
> > I have prepared PR for it https://github.com/php/php-src/pull/9315
>
> I'm not against increasing the sizes, but 50MB might be too much.
>
> Anyway, only changing the php.ini files doesn't make sense in my
> opinion, since they're probably only used on Windows, and they should
> reflect the actual default values[1].
>
>
 It's true that those particular ini files are not directly used on Linux
but distros often base their changes on them and the distro provided ones
are actually used. So it means that main defaults in main.c are most likely
not used. Although I agree they should be changed too so it is consistent.

Regards

Jakub

--0000000000008f135605e827fe47--