Describe the problem you’re having:
Originally reported by Ulrich Spörlein via email, the php filter module missing from sanity or installer checks.
filter_var() is used by include/functions.php validate_url and classes/article.php create_published_article however install/index.php and include/sanity_check.php do not have any check for the presence of the php filter module.
Would it be possible to add them to either sanity or installer checks and Tiny Tiny RSS – PHP requirements?
If possible include steps to reproduce the problem:
Attempt to use create_published_article (bookmarklets plugin) or include/functions.php’s validate_url.
See something like the following in error: “… PHP Fatal error: Uncaught Error: Call to undefined function filter_var() in …”
tt-rss version (including git commit id):
05ef9aac2fe
Platform (i.e. Linux distro, PHP, PostgreSQL, etc) versions:
FreeBSD 12.1, PHP 7.4.10, postgres 11.9
fox
2
stuff like this is why these days i’m pretty much only testing against docker installs.
i can’t realistically put the entirety of php standard library used by tt-rss into sanity_check.php just because some freebsd (already an unsupported platform) user has nothing better to do than build himself special snowflake versions of php with random stuff disabled.
in fact, this makes me think that this list of checks has largely outlived its usefulness, except for maybe hinting (to developer) at what’s missing when deploying tt-rss in a some kind of semi-sane development environment, while everyone else should use containers.
e: i’m not sure how this should be dealt with but i’m fairly certain that adding to sanity check function is not it, a different approach is needed, i think.
