search

web-push-libs / web-push-php

Web Push library for PHP
MIT License
1.69k stars 295 forks source link

gmp extension is required #336

Closed richardpcardoso closed 3 years ago

richardpcardoso commented 3 years ago

Hi.

Readme says: Requirements PHP 7.2+ and the following extensions:

gmp (optional but better for performance)

I have installed and enabled gmp on php 7.4.21. I confirm with php -m|grep gmp.

But still got the error message: [WebPush] gmp extension is not loaded but is required for sending push notifications with payload or for VAPID authentication. You can fix this in your php.ini.

Please can you help?

Dartui commented 3 years ago

In README.md there is statement that gmp is optional, but in code there is hardcoded requirement:

https://github.com/web-push-libs/web-push-php/blob/d87e9e3034ca2b95b1822b1b335e7761c14b89f6/src/WebPush.php#L71-L81