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

Complains of 'mbstring' not loaded. #12

Closed
aptonline opened this issue Dec 3, 2019 · 6 comments
Closed

Complains of 'mbstring' not loaded. #12

aptonline opened this issue Dec 3, 2019 · 6 comments

Comments

@aptonline
Copy link

Hi, I finally got around to testing the new images you made for the path issue I posted but upon using the new images I received an error saying that 'mbstring' was not loaded.

I fixed the issue by installing manually.

su
apt-get update
apt-get install php-mbstring

No sure if you need to check its installed as standard with the images.

@dsavell
Copy link
Owner

dsavell commented Dec 8, 2019

Hey Thanks again,

Are you able to provide the steps on how to repeat this?

@aptonline
Copy link
Author

The mbstring error appeared as soon as I went to the front end url, also showed when I went to the admin login. I downloaded the admin image to replace the one I already had. After I installed mbstring using the above method it all worked as expected.

@Miroka96
Copy link
Contributor

Miroka96 commented Dec 8, 2019

I have had the same error using the dsavell/grav:admin image and was able to fix it by running 'apt install php7.3-mbstring' inside the container and restarting the container

@Miroka96
Copy link
Contributor

Miroka96 commented Dec 8, 2019

To prevent such errors in the future it might be an advantage to make the admin image inherit from the normal image

@dsavell
Copy link
Owner

dsavell commented Dec 9, 2019

Thank both, will take a look as soon as I can, I have raised and issue for inheritance that makes a lot of sense. #14

Will get reviewing and rebuilding asap.

@dsavell
Copy link
Owner

dsavell commented Dec 9, 2019

Merged accepted, should now be resolved in latest tags, please repull

@dsavell dsavell closed this as completed Dec 9, 2019
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

No branches or pull requests

3 participants