Clean hostname/name sent to operator to fix issue of hostname with li…

…nebreaks
metalbear-co · May 29, 2024 · 4c7c408 · 4c7c408
1 parent 857dd47
commit 4c7c408
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/mirrord/operator/src/client.rs b/mirrord/operator/src/client.rs
@@ -514,21 +514,19 @@ impl OperatorApi {
                 .uri(self.connect_url(&session_info))
                 .header("x-session-id", session_info.metadata.session_id.to_string());
 
-            debug!(
-                "session_id {}",
-                session_info.metadata.session_id.to_string()
-            );
+            // we trim name/hostname since we saw users having line breaks and maybe it can have
+            // non ascii stuf???
             if let Some(name) = name {
-                debug!(?name, "name");
-                builder =
-                    builder.header("x-client-name", name.replace(|c: char| !c.is_ascii(), ""));
+                builder = builder.header(
+                    "x-client-name",
+                    name.replace(|c: char| !c.is_ascii(), "").trim(),
+                );
             };
 
             if let Some(hostname) = hostname {
-                debug!(?hostname, "hostname");
                 builder = builder.header(
                     "x-client-hostname",
-                    hostname.replace(|c: char| !c.is_ascii(), ""),
+                    hostname.replace(|c: char| !c.is_ascii(), "").trim(),
                 );
             };