Skip to content

Fix connstate indication#732

Merged
pappz merged 9 commits into
mainfrom
fix-connstate-indication
Mar 16, 2023
Merged

Fix connstate indication#732
pappz merged 9 commits into
mainfrom
fix-connstate-indication

Conversation

@pappz

@pappz pappz commented Mar 9, 2023

Copy link
Copy Markdown
Collaborator

Describe your changes

Fix the status indication in the client service. The status of the
management server and the signal server was incorrect if the network
connection was broken. Basically the status update was not used by
the management and signal library.

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

Comment thread client/server/server.go
@pappz
pappz requested a review from mlsmaycon March 14, 2023 13:20
@braginini
braginini self-requested a review March 15, 2023 13:03
@pappz
pappz merged commit 7477972 into main Mar 16, 2023
@pappz
pappz deleted the fix-connstate-indication branch March 16, 2023 16:22
pulsastrix pushed a commit to pulsastrix/netbird that referenced this pull request Dec 24, 2023
Fix the status indication in the client service. The status of the
management server and the signal server was incorrect if the network
connection was broken. Basically the status update was not used by
the management and signal library.
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