Remove additional logic to set custom node path for containers (#5428)

sanjuyadav24 · Sanju Yadav · web-flow · commit 20668bfd7907 · 2025-12-12T10:14:13.000+05:30
* Remove additional logic to set custom node path for containers

* Remove additional logic to set custom node path for containers

---------

Co-authored-by: Sanju Yadav &lt;sanjuyadav@microsoft.com&gt;
diff --git a/src/Agent.Worker/ContainerOperationProvider.cs b/src/Agent.Worker/ContainerOperationProvider.cs
@@ -1008,19 +1008,6 @@ string useDoubleQuotes(string value)
                             }
                         }
 
-                        if (!container.NeedsNode20Redirect)
-                        {
-                            container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node24Folder, "bin", $"node{IOUtil.ExeExtension}"));
-                        }
-                        else if (!container.NeedsNode16Redirect)
-                        {
-                            container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node20_1Folder, "bin", $"node{IOUtil.ExeExtension}"));
-                        }
-                        else
-                        {
-                            container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node16Folder, "bin", $"node{IOUtil.ExeExtension}"));
-                        }
-
                     }
 
                     if (!string.IsNullOrEmpty(containerUserName))
diff --git a/src/Agent.Worker/ContainerOperationProviderEnhanced.cs b/src/Agent.Worker/ContainerOperationProviderEnhanced.cs
@@ -1189,19 +1189,6 @@ string useDoubleQuotes(string value)
                                     );
                                 }
                             }
-
-                            if (!container.NeedsNode20Redirect)
-                            {
-                                container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node24Folder, "bin", $"node{IOUtil.ExeExtension}"));
-                            }
-                            else if (!container.NeedsNode16Redirect)
-                            {
-                                container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node20_1Folder, "bin", $"node{IOUtil.ExeExtension}"));
-                            }
-                            else
-                            {
-                                container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node16Folder, "bin", $"node{IOUtil.ExeExtension}"));
-                            }
                         }
 
                         if (!string.IsNullOrEmpty(containerUserName))

Original file line number	Diff line number	Diff line change
`@@ -1008,19 +1008,6 @@ string useDoubleQuotes(string value)`
`1008`	`1008`	`}`
`1009`	`1009`	`}`
`1010`	`1010`
`1011`		`- if (!container.NeedsNode20Redirect)`
`1012`		`- {`
`1013`		`- container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node24Folder, "bin", $"node{IOUtil.ExeExtension}"));`
`1014`		`- }`
`1015`		`- else if (!container.NeedsNode16Redirect)`
`1016`		`- {`
`1017`		`- container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node20_1Folder, "bin", $"node{IOUtil.ExeExtension}"));`
`1018`		`- }`
`1019`		`- else`
`1020`		`- {`
`1021`		`- container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node16Folder, "bin", $"node{IOUtil.ExeExtension}"));`
`1022`		`- }`
`1023`		`-`
`1024`	`1011`	`}`
`1025`	`1012`
`1026`	`1013`	`if (!string.IsNullOrEmpty(containerUserName))`
Original file line number	Diff line number	Diff line change
`@@ -1189,19 +1189,6 @@ string useDoubleQuotes(string value)`
`1189`	`1189`	`);`
`1190`	`1190`	`}`
`1191`	`1191`	`}`
`1192`		`-`
`1193`		`- if (!container.NeedsNode20Redirect)`
`1194`		`- {`
`1195`		`- container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node24Folder, "bin", $"node{IOUtil.ExeExtension}"));`
`1196`		`- }`
`1197`		`- else if (!container.NeedsNode16Redirect)`
`1198`		`- {`
`1199`		`- container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node20_1Folder, "bin", $"node{IOUtil.ExeExtension}"));`
`1200`		`- }`
`1201`		`- else`
`1202`		`- {`
`1203`		`- container.ResultNodePath = container.TranslateToContainerPath(Path.Combine(HostContext.GetDirectory(WellKnownDirectory.Externals), NodeHandler.Node16Folder, "bin", $"node{IOUtil.ExeExtension}"));`
`1204`		`- }`
`1205`	`1192`	`}`
`1206`	`1193`
`1207`	`1194`	`if (!string.IsNullOrEmpty(containerUserName))`