-
Notifications
You must be signed in to change notification settings - Fork 497
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
[client] Fix controller re-connection #2758
Merged
+814
−524
Merged
Changes from 85 commits
Commits
Show all changes
86 commits
Select commit
Hold shift + click to select a range
9d75cc3
Add pause function for proxies
pappz acad98e
Code cleaning
pappz 90d9dd4
Remove unused function from eBPF proxy
pappz f18ae8b
Apply pause logic
pappz 48184ec
Fix eBPF pause handling
pappz 718d952
Fix test
pappz 9e7aac3
Reducate cognitive complexity
pappz 1f25bb0
Reducate cognitive complexity
pappz ba3cdb3
Remove unnecessary ctx cancel check
pappz 20815c9
Remove unused function
pappz d294cea
Apply sender for bind proxy
pappz b04d19b
Fix nil pointer in error handling
pappz 2b81a68
Add read loop to proxy
pappz fac8869
Add comment
pappz 1814b07
Replace error check to errors.Is
pappz deeb050
Handle addr resolve error
pappz e3dfbe5
Add trace log
pappz c366ddf
- Add pause logic
pappz 9ea3d24
- decrease the buffer copy
pappz 69b1e9e
Rename userspace proxy to udp proxy
pappz 6369706
Merge branch 'main' into relay/fix/wg-roaming
pappz 0ef7e76
Merge branch 'relay/fix/wg-roaming' into eliminate-udp-proxy
pappz 5b2d5f8
Try to force install libpcap
pappz 021eef5
Fix tests
pappz a5deeda
Revert force install change
pappz b8026ad
Merge branch 'main' into relay/fix/wg-roaming
pappz 8739b2a
Merge branch 'relay/fix/wg-roaming' into eliminate-udp-proxy
pappz 553bd32
Fix Android build in factory
pappz 4d788a8
Fix darwin
pappz 2054944
Fix windows build
pappz 25693e0
Lint fix
pappz 4ac34c8
Use USP proxy on all platform
pappz 53e7f08
Fix test
pappz d62b348
Add freebsd proxy factory
pappz 4325a2d
Sonar lint fix
pappz 7fb10d2
Comments
pappz d64e973
Merge branch 'main' into eliminate-udp-proxy
pappz e191f35
fix merge conflict
pappz 4a21eb5
Store the endpoint info with netip.Addr key
pappz 78ef8a6
Fix remove function
pappz 02f08a5
Remove code duplication
pappz 46d1d1a
Apply new logic
pappz 39f4ace
Remove handshaker from guard code
pappz 9a83fe9
Remove unused code
pappz d2e673f
Rename function
pappz 086845f
Code format engine.go
pappz f387d96
Implement close function for sr_watcher
pappz 83c0761
Fix pkg name
pappz bed5f5b
Fix parameters of test
pappz f332acf
Fix signal moc client
pappz 1203f04
Remove unused codes
pappz 5b43c56
Fix map creation
pappz 4428d00
Fix function call in bg
pappz df74d79
Fix ticker timeout
pappz 17e460e
Ignore unnecessary error logging
pappz cac6ad9
Add log
pappz c60dfba
Fix ticker creation
pappz 01e42c9
Fix hardcoded controller logic
pappz dbb280f
Fix test timeout period
pappz abfea86
Fix service is ready eval
pappz 6d4c068
Code cleaning
pappz 4aeca0e
Code cleaning
pappz 022413e
Fix sonar issue
pappz c78355a
Add fast reconnection try to Relay lib
pappz d9ddd29
Lint fix
pappz 8e38a85
Add comment
pappz 21be242
Remove debug line
pappz 2ee4840
Fix CI test
pappz dbcd221
Fix sonar lint
pappz 277db47
Reduce complexity
pappz 0eb5f7b
Fix test
pappz a9597d1
Fix engine test
pappz f4f0820
Fix iface tests in CI
pappz 29e0ae0
Remove -t from docker cmd
pappz c43f142
Remove -i
pappz c48d331
Try to disable parallel run for iface tests
pappz 0f5c024
Revert parallel test
pappz bf6bf68
Workaround for test
pappz 9b36694
Add log
pappz b44a912
Fix FreeBSD build
pappz 72be6dc
Remove debug log line
pappz 5b01236
Change log messages
pappz 63b0f79
Merge branch 'main' into relay/fix-controller-reconnection
pappz be79bff
Replace to errors.Is
pappz 9f2e238
Merge branch 'main' into relay/fix-controller-reconnection
pappz ce11075
Merge branch 'main' into relay/fix-controller-reconnection
pappz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should these new logs be
Debug
level?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this log line is really important to know in any case. And it will appear only once time. We support 3 different implementation and good to know what is the active.