fix: Memory leak in repo-server (#20876) (#20894)

Signed-off-by: Adam Chandler <[email protected]> Co-authored-by: Adam Chandler <[email protected]>
argoproj · Nov 21, 2024 · f8d6665 · f8d6665
1 parent 0680ddb
commit f8d6665
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/util/app/discovery/discovery.go b/util/app/discovery/discovery.go
@@ -170,6 +170,7 @@ func cmpSupports(ctx context.Context, pluginSockFilePath, appPath, repoPath, fil
 	cfg, err := cmpClient.CheckPluginConfiguration(ctx, &empty.Empty{})
 	if err != nil {
 		log.Errorf("error checking plugin configuration %s, %v", fileName, err)
+		io.Close(conn)
 		return nil, nil, false
 	}
 
@@ -178,6 +179,7 @@ func cmpSupports(ctx context.Context, pluginSockFilePath, appPath, repoPath, fil
 		if namedPlugin {
 			return conn, cmpClient, true
 		}
+		io.Close(conn)
 		return nil, nil, false
 	}