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

Attempted to call an undefined method named "isValidWebhookHMAC" of class "Adyen\Util\HmacSignature"." at Adyen/Service/WebhookReceiver.php line 39 #539

Closed
Phobetor opened this issue Aug 8, 2023 · 1 comment · Fixed by #540

Comments

@Phobetor
Copy link

Phobetor commented Aug 8, 2023

Describe the bug
WebhookReceiver::validateHmac() calls HmacSignature::isValidWebhookHMAC() which does not exist, the method is actually HmacSignature::isValidNotificationHMAC().

To Reproduce
Steps to reproduce the behavior:
Just execute this code. There is not even a real notification required.

$receiver = new \Adyen\Service\WebhookReceiver(new \Adyen\Util\HmacSignature());
$receiver->validateHmac(['pspReference' => ''], '1234');

Expected behavior
The method should return a boolean value.

Additional context
Using version 15.0.0-beta of adyen/php-api-library.

jillingk added a commit that referenced this issue Aug 8, 2023
@jillingk
Copy link
Contributor

jillingk commented Aug 8, 2023

Hi @Phobetor,

Thanks a lot for reaching out to us with this bug! We've indeed made a little mistake there during the renaming from notifications -> webhooks. I created a quick PR resolving this issue, let us know if you run into anything else.

Best, Jilling
Adyen

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

Successfully merging a pull request may close this issue.

2 participants