Skip to content

whatsapp-cloud :tolerate malformed webhook_port on adapter init - #80187

Closed
aydnOktay wants to merge 1 commit into
NousResearch:mainfrom
aydnOktay:fix/whatsapp-cloud-webhook-port-guard
Closed

whatsapp-cloud :tolerate malformed webhook_port on adapter init#80187
aydnOktay wants to merge 1 commit into
NousResearch:mainfrom
aydnOktay:fix/whatsapp-cloud-webhook-port-guard

Conversation

@aydnOktay

Copy link
Copy Markdown
Contributor

Summary

  • WhatsAppCloudAdapter used unguarded int() for webhook_port.
  • A malformed config value raised during construction and took the Cloud webhook adapter down instead of falling back to the documented default port.
  • Add a small int coercion helper and regression coverage for the invalid-port case.

WhatsAppCloudAdapter used unguarded int() for webhook_port. A malformed
config value raised during construction and took the Cloud webhook adapter
down instead of falling back to the documented default port.

Co-authored-by: Cursor <cursoragent@cursor.com>
@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists comp/gateway Gateway runner, session dispatch, delivery platform/whatsapp WhatsApp Business adapter sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades labels Aug 6, 2026
@aydnOktay

Copy link
Copy Markdown
Contributor Author

Closing to concentrate review on a single linked product bug: #89979.

@aydnOktay aydnOktay closed this Aug 19, 2026
@aydnOktay

Copy link
Copy Markdown
Contributor Author

Closing to concentrate review on a single linked product bug: #89979.

1 similar comment
@aydnOktay

Copy link
Copy Markdown
Contributor Author

Closing to concentrate review on a single linked product bug: #89979.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P2 Medium — degraded but workaround exists platform/whatsapp WhatsApp Business adapter sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants