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 connection setup callback #190

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

livius-ungureanu
Copy link

Problem:

The ConnectionActor of the rabbitmq-client library offers the possibility to register a connectionSetup callback that is called every time it manages to connect to RabbitMQ. But op-rabbit currently does not expose such a connectionSetup callback in the RabbitControl constructor so it should as user's of op-rabbit should be allowed to register a connectionSetup callback in order to know when a connection goes up or down.

@timcharper
Copy link
Member

Does using GetConnectionActor not solve the problem for you?

@livius-ungureanu
Copy link
Author

As op-rabbit user I am interested for example on logging an alarm instantly and _asynchronously _ when the RabbitMQ connection goes down(and this can be achieved only by registering the connectionSetup callback as the connectionSetup registering is propagated into the amqp driver).

Using GetConnectionActor , the user would be forced to regularly query the ConnectionActor state which is not acceptable.

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