fix environment toggle bug (usebruno#1932)

jwetzell · Mar 26, 2024 · 1349a79 · 1349a79
1 parent c20beab
commit 1349a79
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/...ents/EnvironmentSettings/EnvironmentList/EnvironmentDetails/EnvironmentVariables/index.js b/...ents/EnvironmentSettings/EnvironmentList/EnvironmentDetails/EnvironmentVariables/index.js
@@ -122,7 +122,7 @@ const EnvironmentVariables = ({ environment, collection }) => {
                 </td>
                 <td>
                   {variable.secret ? (
-                    <div>{maskInputValue(variable.value)}</div>
+                    <div className="overflow-hidden text-ellipsis">{maskInputValue(variable.value)}</div>
                   ) : (
                     <SingleLineEditor
                       theme={storedTheme}

diff --git a/packages/bruno-app/src/utils/collections/index.js b/packages/bruno-app/src/utils/collections/index.js
@@ -640,8 +640,13 @@ export const getAllVariables = (collection) => {
   };
 };
 
-export const maskInputValue = (value) =>
-  value
+export const maskInputValue = (value) => {
+  if (!value || typeof value !== 'string') {
+    return '';
+  }
+
+  return value
     .split('')
     .map(() => '*')
     .join('');
+};