remove legacy call

Author: ncode

outbound.go

@@ -67,7 +67,7 @@ func generateChainName(netName, containerID string) string {
 }
 
 // parseArgs extracts additional outbound rules and metadata from CNI_ARGS.
-func parseArgs(args, containerID string) ([]iptables.OutboundRule, map[string]string, error) {
+func parseArgs(args string) ([]iptables.OutboundRule, map[string]string, error) {
 	logger.Log(context.Background(), slog.LevelInfo,
 		"Parsing CNI arguments",
 		getLogAttrs(),
@@ -234,7 +234,7 @@ func applyAdditionalRules(conf *PluginConf, additionalRules []iptables.OutboundR
 // parseConfig is the main entry for reading stdin config + CNI_ARGS.
 // It delegates to smaller helper functions for clarity.
 func parseConfig(stdin []byte, args, containerID string) (*PluginConf, error) {
-	additionalRules, argsMetadata, err := parseArgs(args, containerID)
+	additionalRules, argsMetadata, err := parseArgs(args)
 	if err != nil {
 		return nil, err
 	}

outbound_test.go

@@ -1555,7 +1555,7 @@ func TestParseAdditionalRules(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			rules, _, err := parseArgs(tc.args, "test-container")
+			rules, _, err := parseArgs(tc.args)
 			if tc.expectedError {
 				assert.Error(t, err)
 			} else {
@@ -1623,7 +1623,7 @@ func TestParseArgsWithLogging(t *testing.T) {
 		t.Run(tc.name, func(t *testing.T) {
 			logBuffer.Reset()
 
-			rules, metadata, err := parseArgs(tc.args, "test-container")
+			rules, metadata, err := parseArgs(tc.args)
 
 			if tc.expectError {
 				assert.Error(t, err)
@@ -1713,7 +1713,7 @@ func TestParseArgs(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			rules, meta, err := parseArgs(tc.args, "test-container")
+			rules, meta, err := parseArgs(tc.args)
 
 			if tc.expectError {
 				assert.Error(t, err)