php / php-tasks

Tasks that need doing. This is for php-src maintainers. The end-user bug tracker is at https://bugs.php.net/.
27 stars 6 forks source link

Review validation of INI settings #31

Open Girgias opened 1 month ago

Girgias commented 1 month ago

We have a lot of INI settings that define their custom error handling (e.g. the OnUpdateSaveHandler function for the session.save_handler INI setting) that could possibly use standard INI functions.

One issue, on which we lack coverage, is if INI settings accept the different INI quantity formats that we support (e.g. using G as a multiplier at the end, or using an explicit binary/octal/hexadecimal prefix).

It may also make sense to add new helper functions to parse INI settings (e.g. one dedicated for file permissions)