bugfix: allow hybrid worker groups with spaces

microsoft · Aug 4, 2023 · 37a42b7 · 37a42b7
1 parent 679aa36
commit 37a42b7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/Providers/nxOMSAutomationWorker/automationworker/3.x/worker/jrdsclient.py b/Providers/nxOMSAutomationWorker/automationworker/3.x/worker/jrdsclient.py
@@ -10,6 +10,7 @@
 import traceback
 
 
+import urllib.parse
 import configuration3 as configuration
 import locallogger
 from workerexception import *
@@ -69,8 +70,9 @@ def get_sandbox_actions(self):
                 }
             ]
         """
+        encoded_hybrid_worker_group_name = urllib.parse.quote(self.HybridWorkerGroupName)
         url = self.base_uri + "/automationAccounts/" + self.account_id + \
-              "/Sandboxes/GetSandboxActions?HybridWorkerGroupName=" + self.HybridWorkerGroupName + \
+              "/Sandboxes/GetSandboxActions?HybridWorkerGroupName=" + encoded_hybrid_worker_group_name + \
               "&api-version=" + self.protocol_version
         response = self.issue_request(lambda u: self.httpClient.get(u), url)
 

diff --git a/Providers/nxOMSAutomationWorker/automationworker/worker/jrdsclient.py b/Providers/nxOMSAutomationWorker/automationworker/worker/jrdsclient.py
@@ -9,6 +9,7 @@
 import traceback
 
 
+import urllib.parse
 import configuration
 import locallogger
 import linuxutil
@@ -68,8 +69,9 @@ def get_sandbox_actions(self):
                 }
             ]
         """
+        encoded_hybrid_worker_group_name = urllib.parse.quote(self.HybridWorkerGroupName)
         url = self.base_uri + "/automationAccounts/" + self.account_id + \
-              "/Sandboxes/GetSandboxActions?HybridWorkerGroupName=" + self.HybridWorkerGroupName + \
+              "/Sandboxes/GetSandboxActions?HybridWorkerGroupName=" + encoded_hybrid_worker_group_name + \
               "&api-version=" + self.protocol_version
         response = self.issue_request(lambda u: self.httpClient.get(u), url)