-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Add Promise to botbuilder-adapter-webex both registerWebhookSubscription functions #2218
base: main
Are you sure you want to change the base?
Conversation
Promisifying should resolve the issue of application breaking on wrong token case |
Thanks for the suggestion. Looks like it will solve the issue. |
The above suggestion is appreciated. It look like it will be the solution to the issue. |
The solution mentioned above seems to resolve the issue. |
It seems like this solution will solve the issue. Appreciated. |
The solution mentioned above seems to resolve the issue.Thanks for the suggestion. |
Thanks for the suggestion. Looks like it will solve the issue. |
Seems like this solution will resolve the issue, Thanks for the suggestion. |
Thanks for the suggestion. Looks like it will solve the issue. |
I think @PunithKrishnamurthy is doing right thing can we get this merged? |
facing similar issue while using this package. Please suggest some solution. |
This is a fix for issue 2217.
The solution wraps the function with promise such that rejects are present and any thrown error can not be easily caught and handled by the user and should not affect existing bots.
Now developers can handle wrong access tokens this way without crashing during webhook subscription:
The changes should fix the issue and be backward compatible too.