sendnrw/netbird
2
0
Fork 0
mirror of https://github.com/netbirdio/netbird.git synced 2026-04-18 08:16:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix missing route

Browse Source
This commit is contained in:
Viktor Liu
2026-02-08 21:36:55 +08:00
parent d2a7f3ae36
commit 2cf00dba58
2 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
management/server/types/account.go
View File

@@ -1928,7 +1928,7 @@ func (a *Account) GetPeerProxyRoutes(ctx context.Context, peer *nbpeer.Peer, pro
for _, proxyPerResource := range proxies {
for _, proxy := range proxyPerResource {
for _, target := range proxy.Targets {
if target.TargetType == reverseproxy.TargetTypeResource && !target.AccessLocal {
if target.TargetType == reverseproxy.TargetTypeResource {
resource, ok := resourcesMap[target.TargetId]
if !ok {
log.WithContext(ctx).Warnf("proxy target %s not found in resources map", target.TargetId)

9
management/server/types/proxy_firewall_rules_test.go
View File

@@ -372,9 +372,12 @@ func TestGetPeerProxyRoutes_ResourceWithAccessLocal(t *testing.T) {
routes, routeFwRules, aclPeers := account.GetPeerProxyRoutes(context.Background(), account.Peers["proxy-peer"], exposedServices, resourcesMap, routers, proxyPeers)
assert.Empty(t, routes, "should NOT generate routes for AccessLocal resource")
assert.Empty(t, routeFwRules, "should NOT generate route firewall rules for AccessLocal resource")
assert.Empty(t, aclPeers, "should NOT include router peer from route path for AccessLocal resource")
require.NotEmpty(t, routes, "should generate routes for AccessLocal resource")
require.NotEmpty(t, routeFwRules, "should generate route firewall rules for AccessLocal resource")
require.NotEmpty(t, aclPeers, "should include router peer in ACL for AccessLocal resource")
assert.Equal(t, uint16(443), routeFwRules[0].PortRange.Start)
assert.Equal(t, "192.168.1.100/32", routeFwRules[0].Destination)
}
func TestGetPeerProxyRoutes_PeerTargetSkipped(t *testing.T) {