xmlrpc_errors INI setting

From: Date: Tue, 28 Apr 2020 19:03:59 +0000
Subject: xmlrpc_errors INI setting
Groups: php.internals 
Request: Send a blank email to internals+get-109893@lists.php.net to get a copy of this message
Hey, while working on cleanups for the internal error handling code, specifically extracting different rendering logic into pluggable hooks ( https://github.com/php/php-src/pull/5484) I came across the ini settings "xmlrpc_errors" and "xmlrpc_error_number", prominently integrated and used in main/main.c Both are used to allow users to change the display rendering of errors while display_errors=1 and these settings were added in 2001 and nothing changed much about this since then. I am wondering if this code is not better moved to ext/xmlrpc extension instead of being in main/main.c - this would be similar to how ext/soap handles errors in the extension already. Downsides: - People can currently use this setting when doing their own xmlrpc, however that anyone uses this setting looks so unlikely, because its global - With the potential unbundling of ext/xmlrpc this could also mean the ini settings get moved out of core entirely. Do you think this small BC break moving the settings to ext/xmlrpc is acceptable as part of decoupling error displaying from php_error_cb and main.c? Has anyone seen this been used in a project?

« previous php.internals (#109893) next »