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

rabbitmq (new) #173

Merged
merged 10 commits into from
Oct 2, 2024
Merged

rabbitmq (new) #173

merged 10 commits into from
Oct 2, 2024

Conversation

DavidDurman
Copy link
Member

No description provided.

@vtalas
Copy link
Contributor

vtalas commented Sep 12, 2024

  • - Publish - when the exchange is defined it throws an error when starting the flow
    image
    The Exchange input is not required, therefore I assuming the default exchange is selected, correct ?

  • - SendToQueue: throws an error when I send a message to the webhook:
    image
    image

it's not always happening, sometimes it succeeds:
image
However, even when the SendToQueue succeeds, the NewMessage is not triggered

@DavidDurman
Copy link
Member Author

No, when the Exchange is empty in Publish (undefined or empty string) - a special case - it will send directly to the queue named by the routing key. That's how the amqplib behaves: https://amqp-node.github.io/amqplib/channel_api.html#channel_publish

@vtalas vtalas self-requested a review September 27, 2024 14:08
@DavidDurman DavidDurman changed the title rabbitmq rabbitmq (new) Sep 27, 2024
@vtalas vtalas merged commit c2bb130 into clientIO:dev Oct 2, 2024
1 check passed
@vtalas vtalas assigned RamBachkheti and unassigned DavidDurman Oct 2, 2024
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.

3 participants