Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:110719
MIME-Version: 1.0
References: <DM6PR07MB6618FF81C67689A8B156F8BAF9950@DM6PR07MB6618.namprd07.prod.outlook.com>
 <CACsECNf4hA=k9dnP3P8bWD7w-sO_mqKMLMgm6KLFsioSzq6RtQ@mail.gmail.com>
In-Reply-To: <CACsECNf4hA=k9dnP3P8bWD7w-sO_mqKMLMgm6KLFsioSzq6RtQ@mail.gmail.com>
Date: Wed, 24 Jun 2020 21:56:42 +0200
Message-ID: <CACsECNdFSAQaNwcqdUS3xavcVkX9w6okmdwyio1UMx6XrFeJdA@mail.gmail.com>
To: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: [PHP-DEV] Making the hardcoded string length limit of
 Throwable->getTraceAsString() configurable
From: alexinbeijing@gmail.com (Alex)

On Wed, Jun 24, 2020 at 9:22 PM tyson andre <tysonandre775@hotmail.com> wrote:
>
> By default, strings in parameter lists are truncated to 15 bytes by default in Throwable->getTraceAsString()
> (and Throwable->__toString() as a consequence).
> (in Zend/zend_exception.c in `static void _build_trace_args(zval *arg, smart_str *str)`)
>
> This limit is too short to see relevant information such as file paths, full urls, etc, which makes reporting bugs in applications inconvenient.
>
> Would there be any interest in an ini setting such as `exception_string_length_limit` as a non-negative value to raise this (either allowed to be 0 or more or 15 or more, defaulting to 15)

Just as a suggestion, might it not be simpler just to increase the
limit to something a bit higher than 15?

(Personally, I view the growth of things like .INI parameters with
suspicion. I feel each one should really "carry its weight". But if
this one does, that is fine.)

Alex