vonage-php-sdk-core icon indicating copy to clipboard operation
vonage-php-sdk-core copied to clipboard

Verify v1 - Deprecations when getting Verification::getRequest()

Open kissifrot opened this issue 2 years ago • 3 comments

We are getting deprecation messages every time we call verify()

Expected Behavior

When using verify()->check($verificationCode, $code), we should not get deprecation warnings

Current Behavior

When using verify()->check($verificationCode, $code), we are getting lots of deprecation messages User Deprecated: Vonage\Verify\Verification::getRequest() is deprecated. Please get the APIResource from the appropriate client to get this information

Possible Solution

Steps to Reproduce (for bugs)

  1. Just use verify()->check($verificationCode, $code) (we use it in symfony context)

Context

Our sentries issues are filled with those messages :sweat_smile:

Your Environment

  • Version used: 4.3.1
  • Environment name and version (e.g. PHP 7.2 on nginx 1.19.1): PHP 8.2 on nginx
  • Operating System and version: Ubuntu 20.04 LTS

kissifrot avatar Aug 29 '23 12:08 kissifrot

Thanks for reporting this. I'll need to do a bit of investigation, as removing the deprecations (we had a circular warning dependency problem here) might be a backward breaking change, which means it'll have to wait until 5.0

SecondeJK avatar Sep 18 '23 09:09 SecondeJK

This still happening.


Vonage\Verify\Verification::getResponse() is deprecated. Please get the APIResource from the appropriate client to get this information in /var/www/html/code/vendor/vonage/client-core/src/Entity/Psr7Trait.php on line 103

Vonage\Verify\Verification::setResponse() is deprecated and will be removed in /var/www/html/code/vendor/vonage/client-core/src/Entity/Psr7Trait.php on line 45

Using Vonage\Verify\Verification for starting a verification is deprecated, please use Vonage\Verify\Request instead in /var/www/html/code/vendor/vonage/client-core/src/Verify/Verification.php on line 83

Creation of dynamic property Vonage\Verify\Verification::$data is deprecated in /var/www/html/code/vendor/vonage/client-core/src/Entity/Psr7Trait.php on line 66

tairoroberto avatar Jul 15 '24 19:07 tairoroberto

This will unfortunately keep happening until version 5 comes out, when deprecations will be actually removed and the internals fixed. I can't change the internal signatures until a major release.

SecondeJK avatar Jul 18 '24 09:07 SecondeJK