Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing mod_php section in .htaccess for PHP8 settings #3252

Closed
addison74 opened this issue May 14, 2023 · 3 comments · Fixed by #3254
Closed

Missing mod_php section in .htaccess for PHP8 settings #3252

addison74 opened this issue May 14, 2023 · 3 comments · Fixed by #3254

Comments

@addison74
Copy link
Contributor

If I remember well we discussed an improved version for the .htaccess file last year. Then a section for PHP8 settings was proposed. Today when I looked in the content of the file we only have specific settings for PHP7, not for PHP8.

Here is what we have at this moment in the file

############################################
## php7 settings

<IfModule mod_php7.c>

############################################
## adjust max execution time

    php_value max_execution_time 18000

############################################
## disable automatic session start
## before autoload was initialized

    php_flag session.auto_start off

############################################
## enable resulting html compression

    #php_flag zlib.output_compression on

###########################################
# disable user agent verification to not break multiple image upload

    php_flag suhosin.session.cryptua off

</IfModule>
@fballiano
Copy link
Contributor

I chcked the history of the .htaccess file but I don't see any change that removed that section, probably we never added it at the end. unless it was in a commit on some branch that got lost.

@fballiano
Copy link
Contributor

it was this PR #2306 anyway

@addison74
Copy link
Contributor Author

The exchange of ideas in that PR showed that improvments are needed in .htaccess. We did the changes for PHP8 in other directories such as media, skin, but not in the root file. We can add a few lines with some important settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants