Merge pull request #79 from telekom-mms/bugfix/splitrt-fixes

Bugfix/splitrt fixes
telekom-mms · Apr 29, 2024 · f481937 · f481937
2 parents dbbaaf2 + cca8c3a
commit f481937
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/internal/splitrt/excludes.go b/internal/splitrt/excludes.go
@@ -104,7 +104,7 @@ func (e *Excludes) AddDynamic(ctx context.Context, address *net.IPNet, ttl uint3
 		return
 	}
 	if !prefix.IsSingleIP() {
-		log.WithError(err).Error("SplitRouting error adding dynamic exclude with multiple IPs")
+		log.Error("SplitRouting error adding dynamic exclude with multiple IPs")
 		return
 	}
 	a := prefix.Addr()

diff --git a/internal/splitrt/splitrt_test.go b/internal/splitrt/splitrt_test.go
@@ -107,7 +107,7 @@ func TestSplitRoutingHandleAddressUpdate(t *testing.T) {
 		t.Errorf("got %v, want %v", got, want)
 	}
 
-	// test with exculde and virtual
+	// test with exclude and virtual
 	vpnconf = vpnconfig.New()
 	vpnconf.Split.ExcludeIPv4 = []*net.IPNet{
 		{
@@ -231,7 +231,7 @@ func TestSplitRoutingStartStop(t *testing.T) {
 		},
 		{
 			IP:   net.ParseIP("2000::1"),
-			Mask: net.CIDRMask(32, 32),
+			Mask: net.CIDRMask(128, 128),
 		},
 	}
 	s = NewSplitRouting(NewConfig(), vpnconf)