On Wed, Jul 1, 2020 at 10:46 PM tyson andre <tysonandre...@hotmail.com> wrote:
> Hi Nikita, > > > Why is the lower limit of the ini setting 15 and not 0? > > > > I just saw that you mention this possibility as "future scope". > > I don't think your reasoning there holds up if you take into account > > that we already introduced zend.exception_ignore_args in PHP 7.4 -- > > and which is enabled by default in production. > > That removes argument information from exceptions completely. > > Setting the string parameter length to 0 still provides more information > than that. > > I wasn't very familiar with what zend.exception_ignore_args did, so I > missed that. > You do have a good point - it may be useful in use cases where string > contents should be completely hidden > but it's still useful to log arg types. > > I updated https://wiki.php.net/rfc/throwable_string_param_max_len - the > minimum is now 0, > and the suggested value in php.ini-production is now 0, and the default > remains at 15. > > ``` > phpdoc-en-git ±master » ag exception_ignore_args > appendices/migration74/other-changes.xml > 34: <literal>zend.exception_ignore_args</literal> is a new INI directive > ``` > > Aside: I think that should be documented in > https://www.php.net/manual/en/errorfunc.configuration.php or somewhere > else. > Right now, the documentation isn't on php.net, just the changelog. Thanks for the update. You'll probably want to remove the corresponding section under future scope now. You may also want to give some more thought about the name of the ini setting name. We already have zend.exception_ignore_args (which is TBH not a great name), and now we get throwable_string_param_max_len, which doesn't follow the same format. Possibly zend.exception_string_param_max_len would fit in better? Nikita
