fix: add mod_php8 IfModule block to .htaccess

[DeepDiver1975]

Summary

PHP 8 registers its Apache module as mod_php8.c, not mod_php7.c. The existing .htaccess only has <IfModule mod_php5.c> and <IfModule mod_php7.c> blocks, so on PHP 8 installs all php_value directives (upload limits, memory limit, charset, output buffering) are silently ignored.

This PR adds a <IfModule mod_php8.c> block mirroring the mod_php7.c block, with one intentional omission: mbstring.func_overload was removed in PHP 8.0 and must not appear in the PHP 8 block.

Changes

• .htaccess: new <IfModule mod_php8.c> block added after the mod_php7.c block, without mbstring.func_overload

Related

Raised in context of #41611 (big file upload configuration / Docker PHP 8 support).

[update-docs]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

[mmattel]

Q: why not removing the outdated other directives? (and moving the block to the bottom?)

[DeepDiver1975]

Adds a <IfModule mod_php8.c> block to .htaccess and removes the now-obsolete mod_php5.c and mod_php7.c blocks.

The fix is correct and the omissions are intentional and well-explained:

• mbstring.func_overload removed in PHP 8.0 — correctly absent from the new block.
• always_populate_raw_post_data was PHP 5 only — correctly absent.
• php_value default_charset and output_buffering are retained — correct.

One thing to verify: mod_php7.c being removed means PHP 7.x installations will silently fall back to compiled-in defaults for upload_max_filesize, post_max_size, memory_limit, etc. If the project still supports PHP 7.4 (Ubuntu 22.04 Docker image does), consider retaining the mod_php7.c block rather than replacing it. The companion PR owncloud-docker/base#472 adds these values to the PHP ini for the Docker image, which covers the Docker case — but bare (non-Docker) PHP 7.x installs relying on .htaccess would regress.

If PHP 7.x is no longer a supported target for this .htaccess, the removal is fine and this is ready to merge.

[phil-davis]

Should this have a changelog?