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

Use of undefined constant SODIUM_LIBRARY_MAJOR_VERSION - assumed 'SODIUM_LIBRARY_MAJOR_VERSION #81

Closed
eaponiente opened this issue Nov 15, 2018 · 3 comments

Comments

@eaponiente
Copy link

PHP: 7.1
Ubuntu: 14.04

followed all the instructions in the website and last part was put sodium.so in my
/etc/php/fpm/php.ini.

now to test it out, I tried the ff code in php artisan tinker(ran in command line) inside one of my laravel projects

var_dump([
    SODIUM_LIBRARY_MAJOR_VERSION,
    SODIUM_LIBRARY_MINOR_VERSION,
    SODIUM_LIBRARY_VERSION
]);

and got this results

array(2) {
  [0] =>
  int(9)
  [1] =>
  int(1)
  [2] =>
  string(6) "1.0.14"
}

Now, when I tried the code on my browser.

I am getting this error Use of undefined constant SODIUM_LIBRARY_MAJOR_VERSION - assumed 'SODIUM_LIBRARY_MAJOR_VERSION

@paragonie-scott
Copy link
Member

I'm not entirely sure what the problem here could be.

Is this a bug with the PHP extension that you intended to report upstream? Or do you want us to expose this constant in the PHP 7.2 version of the polyfill too?

@paragonie-scott
Copy link
Member

If you're looking to fix your PECL configuration, there's not much help I can offer.

But if you wanted the polyfill to include these constants, it does in 1.8.0.

@eaponiente
Copy link
Author

Thanks for the very helpful response. I only needed to restart the server in order for it to work. lol

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

2 participants