Skip to content

Fix: Skip state notification should use a copy of the previous peer state - #960

Merged
mlsmaycon merged 2 commits into
mainfrom
fix/peer-conn-status-use-oldstate
Jun 17, 2023
Merged

Fix: Skip state notification should use a copy of the previous peer state#960
mlsmaycon merged 2 commits into
mainfrom
fix/peer-conn-status-use-oldstate

Conversation

@mlsmaycon

Copy link
Copy Markdown
Collaborator

Describe your changes

This was affecting the behavior of the route manager, causing issues with HA and with cases of flaky connections.

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

…tate

This was affecting the behavior of the route manager,
causing issues with HA and with cases of flaky connections
@mlsmaycon
mlsmaycon requested review from braginini and pappz June 16, 2023 21:32
@mlsmaycon
mlsmaycon merged commit a4350c1 into main Jun 17, 2023
@mlsmaycon
mlsmaycon deleted the fix/peer-conn-status-use-oldstate branch June 17, 2023 07:03
pulsastrix pushed a commit to pulsastrix/netbird that referenced this pull request Dec 24, 2023
…tate (netbirdio#960)

This was affecting the behavior of the route manager,
causing issues with HA and with cases of flaky connections
Foosec pushed a commit to Foosec/netbird that referenced this pull request May 8, 2024
…tate (netbirdio#960)

This was affecting the behavior of the route manager,
causing issues with HA and with cases of flaky connections
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.

2 participants