[Proposal] Make string length for getTraceAsString() configurable

Add a `throwable_string_param_max_len` ini setting.
(same suffix as `log_errors_max_len`)

Allow values between 15 and 1000000. Future RFCs may expand that range.

Discussion: https://externals.io/message/110717

Author: TysonAndre

Zend/tests/exception_024.phpt

@@ -0,0 +1,19 @@
+--TEST--
+throwable_string_param_max_len ini setting
+--INI--
+throwable_string_param_max_len = 23
+--FILE--
+<?php
+
+function main($arg) {
+    throw new Exception();
+}
+main('123456789012345678901234567890');
+
+?>
+--EXPECTF--
+Fatal error: Uncaught Exception in %s:%d
+Stack trace:
+#0 %s(%d): main('12345678901234567890123...')
+#1 {main}
+  thrown in %s on line %d

Zend/tests/exception_025.phpt

@@ -0,0 +1,29 @@
+--TEST--
+throwable_string_param_max_len ini setting
+--FILE--
+<?php
+
+function main($arg) {
+    echo (new Exception()), "\n";
+}
+var_dump(ini_set('throwable_string_param_max_len', '14'));
+var_dump(ini_set('throwable_string_param_max_len', '1000001'));
+var_dump(ini_set('throwable_string_param_max_len', '1000000'));
+var_dump(ini_set('throwable_string_param_max_len', '20'));
+main('short');
+main('123456789012345678901234567890');
+
+?>
+--EXPECTF--
+bool(false)
+bool(false)
+string(2) "15"
+string(7) "1000000"
+Exception in %s:%d
+Stack trace:
+#0 %s: main('short')
+#1 {main}
+Exception in %s:%d
+Stack trace:
+#0 %s: main('12345678901234567890...')
+#1 {main}

Zend/zend_exceptions.c

@@ -28,6 +28,7 @@
 #include "zend_dtrace.h"
 #include "zend_smart_str.h"
 #include "zend_exceptions_arginfo.h"
+#include "php_globals.h"
 
 ZEND_API zend_class_entry *zend_ce_throwable;
 ZEND_API zend_class_entry *zend_ce_exception;
@@ -480,8 +481,8 @@ static void _build_trace_args(zval *arg, smart_str *str) /* {{{ */
 			break;
 		case IS_STRING:
 			smart_str_appendc(str, '\'');
-			smart_str_append_escaped(str, Z_STRVAL_P(arg), MIN(Z_STRLEN_P(arg), 15));
-			if (Z_STRLEN_P(arg) > 15) {
+			smart_str_append_escaped(str, Z_STRVAL_P(arg), MIN(Z_STRLEN_P(arg), PG(throwable_string_param_max_len)));
+			if (Z_STRLEN_P(arg) > PG(throwable_string_param_max_len)) {
 				smart_str_appends(str, "...', ");
 			} else {
 				smart_str_appends(str, "', ");

main/main.c

@@ -264,6 +264,21 @@ static PHP_INI_MH(OnSetSerializePrecision)
 }
 /* }}} */
 
+/* {{{ PHP_INI_MH
+ */
+static PHP_INI_MH(OnSetThrowableStringParamMaxLen)
+{
+	zend_long i;
+
+	ZEND_ATOL(i, ZSTR_VAL(new_value));
+	if (i >= 15 && i <= 1000000) {
+		PG(throwable_string_param_max_len) = i;
+		return SUCCESS;
+	} else {
+		return FAILURE;
+	}
+}
+/* }}} */
 
 /* {{{ PHP_INI_MH
  */
@@ -725,6 +740,7 @@ PHP_INI_BEGIN()
 	STD_PHP_INI_BOOLEAN("register_argc_argv",	"1",		PHP_INI_PERDIR|PHP_INI_SYSTEM,	OnUpdateBool,	register_argc_argv,		php_core_globals,	core_globals)
 	STD_PHP_INI_BOOLEAN("auto_globals_jit",		"1",		PHP_INI_PERDIR|PHP_INI_SYSTEM,	OnUpdateBool,	auto_globals_jit,	php_core_globals,	core_globals)
 	STD_PHP_INI_BOOLEAN("short_open_tag",	DEFAULT_SHORT_OPEN_TAG,	PHP_INI_SYSTEM|PHP_INI_PERDIR,		OnUpdateBool,			short_tags,				zend_compiler_globals,	compiler_globals)
+	STD_PHP_INI_ENTRY("throwable_string_param_max_len",	"15",	PHP_INI_ALL,	OnSetThrowableStringParamMaxLen,	throwable_string_param_max_len,		php_core_globals,	core_globals)
 
 	STD_PHP_INI_ENTRY("unserialize_callback_func",	NULL,	PHP_INI_ALL,		OnUpdateString,			unserialize_callback_func,	php_core_globals,	core_globals)
 	STD_PHP_INI_ENTRY("serialize_precision",	"-1",	PHP_INI_ALL,		OnSetSerializePrecision,			serialize_precision,	php_core_globals,	core_globals)

main/php_globals.h

@@ -167,6 +167,8 @@ struct _php_core_globals {
 	char *syslog_ident;
 	zend_bool have_called_openlog;
 	zend_long syslog_filter;
+
+	zend_long throwable_string_param_max_len;
 };
 
 

run-tests.php

@@ -340,6 +340,7 @@ function main()
         'zend.assertions=1',
         'zend.exception_ignore_args=0',
         'short_open_tag=0',
+        'throwable_string_param_max_len=15',
     );
 
     $no_file_cache = '-d opcache.file_cache= -d opcache.file_cache_only=0';