Replace invalid characters with underscore instead of removing them

Copilot · TingluoHuang · Copilot · commit 135f4fc944db · 2026-01-06T22:39:03.000Z
Co-authored-by: TingluoHuang &lt;1750815+TingluoHuang@users.noreply.github.com&gt;
diff --git a/dist/index.js b/dist/index.js
@@ -36325,8 +36325,8 @@ function getUserAgentWithOrchestrationId(userAgent) {
     if (!orchestrationId) {
         return userAgent;
     }
-    // Sanitize orchestration ID - only keep alphanumeric, dots, hyphens, and underscores
-    const sanitized = orchestrationId.replace(/[^a-zA-Z0-9._-]/g, '');
+    // Sanitize orchestration ID - replace invalid characters with underscore
+    const sanitized = orchestrationId.replace(/[^a-zA-Z0-9._-]/g, '_');
     if (!sanitized) {
         return userAgent;
     }
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/main.ts b/src/main.ts
@@ -111,8 +111,8 @@ function getUserAgentWithOrchestrationId(userAgent: string): string {
     return userAgent
   }
 
-  // Sanitize orchestration ID - only keep alphanumeric, dots, hyphens, and underscores
-  const sanitized = orchestrationId.replace(/[^a-zA-Z0-9._-]/g, '')
+  // Sanitize orchestration ID - replace invalid characters with underscore
+  const sanitized = orchestrationId.replace(/[^a-zA-Z0-9._-]/g, '_')
   if (!sanitized) {
     return userAgent
   }

Original file line number	Diff line number	Diff line change
`@@ -36325,8 +36325,8 @@ function getUserAgentWithOrchestrationId(userAgent) {`
`36325`	`36325`	`if (!orchestrationId) {`
`36326`	`36326`	`return userAgent;`
`36327`	`36327`	`}`
`36328`		`- // Sanitize orchestration ID - only keep alphanumeric, dots, hyphens, and underscores`
`36329`		`- const sanitized = orchestrationId.replace(/[^a-zA-Z0-9._-]/g, '');`
	`36328`	`+ // Sanitize orchestration ID - replace invalid characters with underscore`
	`36329`	`+ const sanitized = orchestrationId.replace(/[^a-zA-Z0-9._-]/g, '_');`
`36330`	`36330`	`if (!sanitized) {`
`36331`	`36331`	`return userAgent;`
`36332`	`36332`	`}`
Original file line number	Diff line number	Diff line change
`@@ -111,8 +111,8 @@ function getUserAgentWithOrchestrationId(userAgent: string): string {`
`111`	`111`	`return userAgent`
`112`	`112`	`}`
`113`	`113`
`114`		`- // Sanitize orchestration ID - only keep alphanumeric, dots, hyphens, and underscores`
`115`		`- const sanitized = orchestrationId.replace(/[^a-zA-Z0-9._-]/g, '')`
	`114`	`+ // Sanitize orchestration ID - replace invalid characters with underscore`
	`115`	`+ const sanitized = orchestrationId.replace(/[^a-zA-Z0-9._-]/g, '_')`
`116`	`116`	`if (!sanitized) {`
`117`	`117`	`return userAgent`
`118`	`118`	`}`