extern/netbird
1
0
Fork 0
mirror of https://github.com/netbirdio/netbird.git synced 2025-06-20 17:58:02 +02:00
Code Issues Packages Projects Releases Wiki Activity

Add close test for all implementation

Browse Source
This commit is contained in:
Zoltán Papp 2025-02-17 21:47:34 +01:00
parent 360c7134f7
commit 1963644c99
2 changed files with 31 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
client/iface/wgproxy/bind/proxy.go
View File

@ -110,6 +110,10 @@ func (p *ProxyBind) CloseConn() error {
} }
func (p *ProxyBind) close() error { func (p *ProxyBind) close() error {
if p.remoteConn == nil {
return nil
}
p.closeMu.Lock() p.closeMu.Lock()
defer p.closeMu.Unlock() defer p.closeMu.Unlock()
@ -125,7 +129,7 @@ func (p *ProxyBind) close() error {
p.pausedCond.L.Unlock() p.pausedCond.L.Unlock()
p.pausedCond.Signal() p.pausedCond.Signal()
p.bind.RemoveEndpoint(bind.EndpointToUDPAddr(*p.wgCurrentUsed)) p.bind.RemoveEndpoint(bind.EndpointToUDPAddr(*p.wgRelayedEndpoint))
if rErr := p.remoteConn.Close(); rErr != nil && !errors.Is(rErr, net.ErrClosed) { if rErr := p.remoteConn.Close(); rErr != nil && !errors.Is(rErr, net.ErrClosed) {
return rErr return rErr

26
client/iface/wgproxy/proxy_test.go
View File

@ -7,14 +7,18 @@ import (
"net" "net"
"testing" "testing"
"github.com/netbirdio/netbird/client/iface/bind"
bindproxy "github.com/netbirdio/netbird/client/iface/wgproxy/bind"
"github.com/netbirdio/netbird/client/iface/wgproxy/ebpf" "github.com/netbirdio/netbird/client/iface/wgproxy/ebpf"
"github.com/netbirdio/netbird/client/iface/wgproxy/udp" "github.com/netbirdio/netbird/client/iface/wgproxy/udp"
"github.com/netbirdio/netbird/util" "github.com/netbirdio/netbird/util"
) )
func init() { func init() {
_ = util.InitLog("debug", "console") _ = util.InitLog("debug", "console")
} }
func TestProxyRedirect(t *testing.T) { func TestProxyRedirect(t *testing.T) {
ebpfProxy := ebpf.NewWGEBPFProxy(51831) ebpfProxy := ebpf.NewWGEBPFProxy(51831)
if err := ebpfProxy.Listen(); err != nil { if err := ebpfProxy.Listen(); err != nil {
@ -31,6 +35,7 @@ func TestProxyRedirect(t *testing.T) {
name string name string
proxy Proxy proxy Proxy
wgPort int wgPort int
endpointAddr *net.UDPAddr
}{ }{
{ {
name: "ebpf kernel proxy", name: "ebpf kernel proxy",
@ -45,12 +50,12 @@ func TestProxyRedirect(t *testing.T) {
} }
for _, tt := range tests { for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) { t.Run(tt.name, func(t *testing.T) {
redirectTraffic(t, tt.proxy, tt.wgPort) redirectTraffic(t, tt.proxy, tt.wgPort, tt.endpointAddr)
}) })
} }
} }
func redirectTraffic(t *testing.T, proxy Proxy, wgPort int) { func redirectTraffic(t *testing.T, proxy Proxy, wgPort int, endPointAddr *net.UDPAddr) {
t.Helper() t.Helper()
msgHelloFromRelay := []byte("hello from relay") msgHelloFromRelay := []byte("hello from relay")
@ -82,7 +87,7 @@ func redirectTraffic(t *testing.T, proxy Proxy, wgPort int) {
_ = relayedServer.Close() _ = relayedServer.Close()
}() }()
if err := proxy.AddTurnConn(context.Background(), nil, relayedConn); err != nil { if err := proxy.AddTurnConn(context.Background(), endPointAddr, relayedConn); err != nil {
t.Errorf("error: %v", err) t.Errorf("error: %v", err)
} }
defer func() { defer func() {
@ -134,7 +139,7 @@ func redirectTraffic(t *testing.T, proxy Proxy, wgPort int) {
} }
} }
func TestProxyCloseByRemoteConnEBPF(t *testing.T) { func TestProxyCloseByRemoteConn(t *testing.T) {
ctx := context.Background() ctx := context.Background()
ebpfProxy := ebpf.NewWGEBPFProxy(51831) ebpfProxy := ebpf.NewWGEBPFProxy(51831)
@ -148,9 +153,15 @@ func TestProxyCloseByRemoteConnEBPF(t *testing.T) {
} }
}() }()
iceBind := bind.NewICEBind(nil, nil)
endpointAddress := &net.UDPAddr{
IP: net.IPv4(10, 0, 0, 1),
Port: 1234,
}
tests := []struct { tests := []struct {
name string name string
proxy Proxy proxy Proxy
endpointAddress *net.UDPAddr
}{ }{
{ {
name: "ebpf proxy", name: "ebpf proxy",
@ -160,6 +171,11 @@ func TestProxyCloseByRemoteConnEBPF(t *testing.T) {
name: "udp proxy", name: "udp proxy",
proxy: udp.NewWGUDPProxy(51832), proxy: udp.NewWGUDPProxy(51832),
}, },
{
name: "bind proxy",
proxy: bindproxy.NewProxyBind(iceBind),
endpointAddress: endpointAddress,
},
} }
relayedConn, _ := net.Dial("udp", "127.0.0.1:1234") relayedConn, _ := net.Dial("udp", "127.0.0.1:1234")
@ -168,7 +184,7 @@ func TestProxyCloseByRemoteConnEBPF(t *testing.T) {
}() }()
for _, tt := range tests { for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) { t.Run(tt.name, func(t *testing.T) {
err := tt.proxy.AddTurnConn(ctx, nil, relayedConn) err := tt.proxy.AddTurnConn(ctx, endpointAddress, relayedConn)
if err != nil { if err != nil {
t.Errorf("error: %v", err) t.Errorf("error: %v", err)
} }