frp/server/group
fatedier c7ac12ea0f
server/group: refactor with shared abstractions and fix concurrency issues (#5222)
* server/group: refactor group package with shared abstractions and fix concurrency issues

Extract common patterns into reusable components:
- groupRegistry[G]: generic concurrent map for group lifecycle management
- baseGroup: shared plumbing for listener-based groups (TCP, HTTPS, TCPMux)
- Listener: unified virtual listener replacing 3 identical implementations

Fix concurrency issues:
- Stale-pointer race: isCurrent check + errGroupStale + controller retry loops
- Worker generation safety: pass realLn and acceptCh as params instead of reading mutable fields
- Connection leak: close conn on worker panic recovery path
- ABBA deadlock in HTTP UnRegister: consistent lock ordering (group.mu -> registry.mu)
- Round-robin overflow in HTTPGroup: use unsigned modulo

Add unit tests (17 tests) for registry, listener, and baseGroup.
Add TCPMux group load balancing e2e test.

* server/group: replace tautological assertion with require.NotPanics

* server/group: remove blank line between doc comment and type declaration
2026-03-08 18:57:21 +08:00
..
base.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
base_test.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
group.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
http.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
https.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
listener.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
listener_test.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
registry.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
registry_test.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
tcp.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00
tcpmux.go server/group: refactor with shared abstractions and fix concurrency issues (#5222) 2026-03-08 18:57:21 +08:00