blockvis / civic-sip-php

Add Civic authorization to any PHP application. Framework agnostic PHP client for Civic Secure Identity Platform.
https://blockvis.com
MIT License
11 stars 3 forks source link

Update composer.json #2

Closed gulaandrij closed 6 years ago

gulaandrij commented 6 years ago

fix php7.2

Zorato commented 6 years ago

Could you please provide more details on how this helps to use php 7.2? I have checked the spomky-labs/jose v6.1.7 requirements, and it has php ^5.6|^7.0.

gulaandrij commented 6 years ago

spomky-labs/jose v6.1.7 has dependent "mdanter/ecc": "0.4.*" which has generate error on php 7.2

Uncaught PHP Exception Symfony\Component\Debug\Exception\FatalErrorException: "Compile Error: Cannot use FG\ASN1\Object as Object because 'Object' is a special class name" at /var/www/git-symfony/vendor/mdanter/ecc/src/Serializer/PrivateKey/DerPrivateKeySerializer.php line 5 {"exception":"[object] (Symfony\Component\Debug\Exception\FatalErrorException(code: 0): Compile Error: Cannot use FG\ASN1\Object as Object because 'Object' is a special class name at /var/www/git-symfony/vendor/mdanter/ecc/src/Serializer/PrivateKey/DerPrivateKeySerializer.php:5)"}

gulaandrij commented 6 years ago

"mdanter/ecc": "0.4.*" https://github.com/phpecc/phpecc/blob/0.4/src/Serializer/PrivateKey/DerPrivateKeySerializer.php#L5

"mdanter/ecc": "0.5.*" https://github.com/phpecc/phpecc/blob/master/src/Serializer/PrivateKey/DerPrivateKeySerializer.php#L6

gulaandrij commented 6 years ago

ping @Zorato

cheelahim commented 6 years ago

Hey @gulaandrij. We have planned to drop supporting PHP 5.6 this year, but we would like too keep for at least couple of months. May be we could change it to "spomky-labs/jose": "^6.1|^7.0" to keep compatibility. If it works for you update the PR and I will merge it and release.

gulaandrij commented 6 years ago

done

cheelahim commented 6 years ago

Thank you for contribution @gulaandrij. Feel free to report any issues.