Skip to content

Conversation

@jjank
Copy link
Contributor

@jjank jjank commented Sep 12, 2024

Follow-up of the discussion Make it easier to detect blocked connections #1415 in rabbitmq-java-client this PR adds a customizer callback for com.rabbitmq.client.Connection(s) akin to the customizer callback for com.rabbitmq.client.ConnectionFactory

This gives users of the JMS-client the opportunity to customize the
established AMQP connection and e.g. add a `BlockedListener` to the
connection.

See rabbitmq/rabbitmq-java-client#1415
@michaelklishin michaelklishin merged commit f05677b into rabbitmq:main Sep 12, 2024
1 check passed
@michaelklishin michaelklishin added this to the 3.3.0 milestone Sep 12, 2024
@michaelklishin
Copy link
Contributor

@acogoluegnes we have last produced a release in Oct 2023, sounds like we should cut a 3.3.0 soon.

github-actions bot pushed a commit that referenced this pull request Sep 12, 2024
Add customizer callback for AMQP connections
@jjank
Copy link
Contributor Author

jjank commented Sep 12, 2024

@michaelklishin any chance this can be included in the 2.x version as well? Asking for a friend 😬

@jjank jjank deleted the add-amqp-connection-customizer branch September 12, 2024 15:47
@michaelklishin
Copy link
Contributor

If it turns out to be easy to backport, I have no objections.

jjank added a commit to jjank/rabbitmq-jms-client that referenced this pull request Sep 12, 2024
This gives users of the JMS-client the opportunity to customize the
established AMQP connection and e.g. add a `BlockedListener` to the
connection.

See rabbitmq/rabbitmq-java-client#1415

Backport of rabbitmq#496
jjank added a commit to jjank/rabbitmq-jms-client that referenced this pull request Sep 12, 2024
This gives users of the JMS-client the opportunity to customize the
established AMQP connection and e.g. add a `BlockedListener` to the
connection.

See rabbitmq/rabbitmq-java-client#1415

Backport of rabbitmq#496
@jjank
Copy link
Contributor Author

jjank commented Sep 12, 2024

Turns out: Yes - I created a PR #498

@acogoluegnes
Copy link
Contributor

Thanks!

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