James Seconde
James Seconde
This can be merged when it is time to release 5.0.0 (as although it's a fix, it's a semver breaking change in line with how the Record NCCO works currently)
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,...
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.
I've just picked up an internal ticket and fixed this in a different PR, although the solution seems to be much the same. Thanks for opening it though! #493
The test suite passes, but codecov is blocking this from being merged. I can look at a way to increase coverage if needed, but is it possible to look at...
New releases in this library and vonage-laravel have been cut - closing.
Would be closed by #92
`Signature` takes an array, so cast the incoming object to an array (I assume right now it's a stdObject?) ``` $payload = json_decode($data, true, 512, JSON_THROW_ON_ERROR); $signature = new \Vonage\Client\Signature($payload,...
The nested data makes sense - I'm reopening and have asked the question internally with the engineering team to see what they come back with. It -might- be the case...
Hi @manavo Unfortunately, this isn't going to be the response you want, but after talking it through with engineering it seems there are some nuances about this particular webhook that...