eclipse / kapua

Eclipse Public License 2.0
222 stars 160 forks source link

:fix: add auto reconnect to service client #4012

Closed riccardomodanese closed 5 months ago

riccardomodanese commented 5 months ago

Brief description of the PR. Added a connection retry when the conection is dropped.

Related Issue fixes 4011

Description of the solution adopted Like for the service event connection a connection retry was added. This is triggered, through the connection event listener, when the connection is dropped.

Screenshots none

Any side note on the changes made none

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 0% with 58 lines in your changes are missing coverage. Please review.

Project coverage is 16.86%. Comparing base (81a090d) to head (f2243bc). Report is 6 commits behind head on develop.

:exclamation: Current head f2243bc differs from pull request most recent head fc8ef49. Consider uploading reports for the commit fc8ef49 to get more accurate results

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/eclipse/kapua/pull/4012/graphs/tree.svg?width=650&height=150&src=pr&token=1P4N4CApH8&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse)](https://app.codecov.io/gh/eclipse/kapua/pull/4012?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse) ```diff @@ Coverage Diff @@ ## develop #4012 +/- ## ============================================= - Coverage 16.87% 16.86% -0.01% Complexity 10 10 ============================================= Files 1989 1987 -2 Lines 51791 51810 +19 Branches 4420 4421 +1 ============================================= Hits 8739 8739 - Misses 42648 42667 +19 Partials 404 404 ``` | [Files](https://app.codecov.io/gh/eclipse/kapua/pull/4012?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse) | Coverage Δ | | |---|---|---| | [...ua/client/security/ServiceClientMessagingImpl.java](https://app.codecov.io/gh/eclipse/kapua/pull/4012?src=pr&el=tree&filepath=client%2Fsecurity%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fclient%2Fsecurity%2FServiceClientMessagingImpl.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y2xpZW50L3NlY3VyaXR5L3NyYy9tYWluL2phdmEvb3JnL2VjbGlwc2Uva2FwdWEvY2xpZW50L3NlY3VyaXR5L1NlcnZpY2VDbGllbnRNZXNzYWdpbmdJbXBsLmphdmE=) | `0.00% <ø> (ø)` | | | [...se/kapua/commons/event/jms/JMSServiceEventBus.java](https://app.codecov.io/gh/eclipse/kapua/pull/4012?src=pr&el=tree&filepath=commons%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fevent%2Fjms%2FJMSServiceEventBus.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvZXZlbnQvam1zL0pNU1NlcnZpY2VFdmVudEJ1cy5qYXZh) | `19.21% <0.00%> (ø)` | | | [...broker/artemis/plugin/security/SecurityPlugin.java](https://app.codecov.io/gh/eclipse/kapua/pull/4012?src=pr&el=tree&filepath=broker%2Fartemis%2Fplugin%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fbroker%2Fartemis%2Fplugin%2Fsecurity%2FSecurityPlugin.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-YnJva2VyL2FydGVtaXMvcGx1Z2luL3NyYy9tYWluL2phdmEvb3JnL2VjbGlwc2Uva2FwdWEvYnJva2VyL2FydGVtaXMvcGx1Z2luL3NlY3VyaXR5L1NlY3VyaXR5UGx1Z2luLmphdmE=) | `0.00% <0.00%> (ø)` | | | [...lipse/kapua/client/security/amqpclient/Client.java](https://app.codecov.io/gh/eclipse/kapua/pull/4012?src=pr&el=tree&filepath=client%2Fsecurity%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fclient%2Fsecurity%2Famqpclient%2FClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y2xpZW50L3NlY3VyaXR5L3NyYy9tYWluL2phdmEvb3JnL2VjbGlwc2Uva2FwdWEvY2xpZW50L3NlY3VyaXR5L2FtcXBjbGllbnQvQ2xpZW50LmphdmE=) | `0.00% <0.00%> (ø)` | |