infinyon / http-source-connector

Official Infinyon HTTP Source Connector
https://www.fluvio.io/connectors/inbound/http/
Apache License 2.0
7 stars 7 forks source link

Bugfix: Accept Option<Vec<String>> for subscription_messages #249

Closed benmanns closed 1 month ago

benmanns commented 1 month ago

This is necessary to avoid an error on passing a websocket_config without providing a subscription_messages value. Without websocket_config, this is currently working fine, but if websocket_config is not None, then there's an error about subscription_messages missing.

Apologies, this was a later change that didn't get caught by the tests.