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

[relay-client] Fix reconnection #3250

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

[relay-client] Fix reconnection #3250

wants to merge 2 commits into from

Conversation

pappz
Copy link
Contributor

@pappz pappz commented Jan 29, 2025

Describe your changes

After we started to allow the agent to run without an established Relay connection the reconnection and connection definition has been changed. In this PR I fix these incosistency.

  • The client structure does not manage the reconnection callback anymore
  • If the quick reconnection (to the original home server) is success we will trigger the onReconnection callback

Issue ticket number and link

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

@pappz pappz changed the title [relay-client] fix reconnection [relay-client] Fix reconnection Jan 30, 2025
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.

1 participant