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

Fix hook endpoint and verify token issues. #31

Merged
merged 3 commits into from
May 13, 2024

Conversation

alejamp
Copy link
Contributor

@alejamp alejamp commented May 12, 2024

Fix hook endpoint, now is working, there was a wrong reference to self.instance then I did improve a bit the logging and handling errors.

The method incorrectly compared the verification token with the WhatsApp token.
To solve this, a new parameter was added to the WhatsApp constructor called verify_token.
So verify endpoint was modified to compare with self.verifytoken

fixed a reference error in endpoint hook.
``` self.instance.xxxx ``` references.

Improve error handling and logging
The method incorrectly compared the verification token with the WhatsApp token.
To solve this, a new parameter was added to the WhatsApp constructor called verify_token.
The endpoint was modified to compare with that token.
Add requests and python-dotenv missing package requirements
@filipporomani
Copy link
Owner

filipporomani commented May 13, 2024

Thanks a lot for your PR :)

@filipporomani filipporomani merged commit 60371dc into filipporomani:main May 13, 2024
3 checks passed
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 this pull request may close these issues.

2 participants