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

Libsodium 1.0.13 or higher required for Magento 2.3.2 #52

Closed
erfanimani opened this issue Jul 7, 2019 · 7 comments
Closed

Libsodium 1.0.13 or higher required for Magento 2.3.2 #52

erfanimani opened this issue Jul 7, 2019 · 7 comments

Comments

@erfanimani
Copy link

I'm getting the following error when installing Magento 2.3.2:

In ErrorHandler.php line 61:
                                                                                                                                                                         
Warning: Use of undefined constant SODIUM_CRYPTO_PWHASH_ALG_ARGON2ID13 - assumed 'SODIUM_CRYPTO_PWHASH_ALG_ARGON2ID13' (this will throw an Error in a future version of PHP) in /var/www/html/vendor/magento/framework/Encryption/Encryptor.php on line 153                                                                                         

Also see magento/magento2#23405

@erfanimani erfanimani changed the title Libsodium required for Magento 2.3.2 Libsodium 1.0.13 or higher required for Magento 2.3.2 Jul 7, 2019
@grll
Copy link

grll commented Jul 8, 2019

You can solve it by adding/updating libsodium library. The following command added to the phpfpm Dockerfile will do:

USER root

RUN curl -O https://download.libsodium.org/libsodium/releases/libsodium-1.0.18.tar.gz \
  && tar xfvz libsodium-1.0.18.tar.gz \
  && cd libsodium-1.0.18 \
  && ./configure \
  && make && make install \
  && pecl install -f libsodium

USER app:app

@erfanimani
Copy link
Author

Thanks, that would be this file: https://github.com/ModestCoders/dockerfiles/blob/master/php/7.2-fpm/Dockerfile

Will create PR when I get the chance.

@tarasyyyk
Copy link

Same problem for me. Where should I create this Dockerfile?

@danielozano
Copy link
Member

danielozano commented Aug 1, 2019

Hi sorry for the late response!
I've already updated php images! Also 7.1 includes sodium library.
You can use the tags 7.2-fpm or 7.2-fpm-2, as explained into #54 the simplified image tags will contain the latest version!

You can check the Dockerfile: https://github.com/ModestCoders/dockerfiles/blob/master/php/7.2-fpm/Dockerfile

Please be aware that until #54 is merged the setup command will not create the php service with the latest image!

Just in case, all the available tags can be found here https://cloud.docker.com/u/modestcoders/repository/docker/modestcoders/php/tags

Test them and let me know if it works!

@tarasyyyk
Copy link

@danielozano I have a problem with composer install. As you can see standard configuration copies files to wrong place:

dockergento composer install

vendor -> phpfpm:vendor
validating and sanitizing path: 'vendor'
removing destination dir content: 'phpfpm:./vendor/*'
ensure destination dir exists: './vendor'
copying './vendor/.' into 'phpfpm:/var/www/html/./vendor'
setting permissions: chown -R app:app /var/www/html/./vendor
Host mirrored into container
Loading composer repositories with package information
Updating dependencies (including require-dev)
Nothing to install or update

[ErrorException]
unlink(./composer.lock): Device or resource busy

As you can see it copiing to /var/www/html/./vendor, not to /var/www/html/vendor.
I didn't change anything, only dockerize my project.

@danielozano
Copy link
Member

@tarasyyyk such problem is not related to this issue, and for sure is related to #53 where I already answer you!

Thanks!

@danielozano
Copy link
Member

Closing it because should be solved with #54, please update the image of your containers!
If the problem persists please feel free to reopen it.

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

4 participants