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

Allow RabbitMq IConnectionFactory override #71

Merged
merged 1 commit into from
May 9, 2022

Conversation

tomaslegenvisma
Copy link
Contributor

Override of RabbitMq IConnectionFactory allows to:

  • set specific ConnectionFactory properties like "AutomaticRecoveryEnabled" for auto-reconnect on connection drop
  • access the connection itself for monitoring health through "IsOpen" property and events like "ConnectionBlocked" and "ConnectionShutdown"

Copy link
Contributor

@jarleborsheim jarleborsheim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍
Godt forslag. Vi skal innføre dette på java-klienten også.
Beklager sent svar.
Vi kommer også til å sette AutomaticRecoveryEnabled til true som default for denne .net klienten.
Det er gjort slik på java-klienten.

@jarleborsheim jarleborsheim merged commit 45fb6f1 into ks-no:master May 9, 2022
@jarleborsheim
Copy link
Contributor

jarleborsheim commented May 9, 2022

@tomaslegenvisma AutomaticRecoveryEnabled skal være default true så da lar vi det bare være slik det er nå 👍

@jarleborsheim
Copy link
Contributor

jarleborsheim commented May 10, 2022

@tomaslegenvisma Etter litt nøyere gjennomgang av helheten i klienten ser vi at det ikke er nødvendigvis bra å kunne sende inn en ConnectionFactory slik det er gjort her. Det bryter da med det klienten gjør med ConnectionFactory, som f.eks. opprette Connection og sette username/password. AutomaticRecoveryEnabled skal være default true. Hvis dere ønsker å monitorere health på connection så må det gjøres på en annen måte. AutomaticRecoveryEnabled skal default være satt til true.

Vi ser gjerne på monitorerings muligheter sammen med dere.

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.

None yet

2 participants