NexmoServiceProvider.php

nat-mystudiosessions commented 3 years ago

Thanks so much for this great tool! This error stumped me for a good while. The way I was reading the error I thought it wasn't getting either the api_secret or the signature_secret. I knew I was providing the api_secret so I wasn't understanding what the problem could be. After a bit of head scratching and digging I found this line in the code and realized that I was providing both the api_secret and the signature_secret values in my .env file, I didn't know that would cause a problem. When I removed the signature_secret value the code worked perfectly. In conclusion, this is just an updated error message to help the next person understand what they're doing wrong. Thanks again.

Description

Motivation and Context

How Has This Been Tested?

Example Output or Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

driesvints commented 2 years ago

@nat-mystudiosessions added this to my PR at https://github.com/laravel/nexmo-notification-channel/pull/53

SecondeJK commented 1 year ago

This package is officially abandoned in favour of https://github.com/Vonage/vonage-laravel. Please use that, and open tickets in that repo if required, thanks.

Nexmo / nexmo-laravel