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

More install/upgrade checks needed? (fileinfo) #7175

Closed
sbulen opened this issue Dec 6, 2021 · 1 comment · Fixed by #7217
Closed

More install/upgrade checks needed? (fileinfo) #7175

sbulen opened this issue Dec 6, 2021 · 1 comment · Fixed by #7217

Comments

@sbulen
Copy link
Contributor

sbulen commented Dec 6, 2021

Description

You get a pretty hard fail if you do not have fileinfo enabled in php.ini. I thought we had a check for that, but apparently not.

We should ensure the upgrader checks are consistent, too. Double-check that mbstring is checked in installer & upgrader as well.

Forum discussion:
https://www.simplemachines.org/community/index.php?topic=579810.0

Steps to reproduce

  1. Install current version. Update avatar in a profile, specifying using an avatar by URL.

Environment (complete as necessary)

  • Version/Git revision: Current
  • Database Type: mysql
  • Database Version: 8.0.20
  • PHP Version: 8.0.13

Additional information/references

If fileinfo is disabled, you white screen with:
image

@sbulen
Copy link
Contributor Author

sbulen commented Dec 29, 2021

I really don't think this is a 2.1.0 requirement. Could we remove or change the milestone?

Note this is already listed in the install requirements.

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

Successfully merging a pull request may close this issue.

2 participants