From e96f799b5b192571b24075202a52751dbdf0db5b Mon Sep 17 00:00:00 2001
From: Sebastian Stamm <sebastian.stamm@camunda.com>
Date: Wed, 27 Sep 2023 10:36:55 +0200
Subject: [PATCH] fix: handle variable and secret values as text

---
 src/main/resources/public/js/incident-modal.js      |  5 ++---
 src/main/resources/public/js/view-connectors.js     |  7 +++++--
 .../resources/public/js/view-process-instance.js    | 13 +++++--------
 3 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/src/main/resources/public/js/incident-modal.js b/src/main/resources/public/js/incident-modal.js
index 5fdc7eae..d67e50b9 100644
--- a/src/main/resources/public/js/incident-modal.js
+++ b/src/main/resources/public/js/incident-modal.js
@@ -113,8 +113,7 @@ async function confirmResolveIncidentModal() {
 
       showNotificationSuccess(
         "set-variables-" + key,
-        "Variables set successfully",
-        "<code>" + variables + "</code>"
+        "Variables set successfully"
       );
 
       track?.("zeebePlay:single-operation", {
@@ -127,7 +126,7 @@ async function confirmResolveIncidentModal() {
     .fail(
       showFailure(
         "set-variables" + scope,
-        "Failed to set variables <code>" + variables + "</code>."
+        "Failed to set variables."
       )
     );
 }
diff --git a/src/main/resources/public/js/view-connectors.js b/src/main/resources/public/js/view-connectors.js
index 4fe11036..767ef32a 100644
--- a/src/main/resources/public/js/view-connectors.js
+++ b/src/main/resources/public/js/view-connectors.js
@@ -22,9 +22,9 @@ function loadConnectorSecrets() {
       let row = `
         <tr>
           <td>${index + 1}</td>
-          <td>${secret.name}</td>
+          <td class="secretName"></td>
           <td>
-            <code>${secret.value}</code>
+            <code></code>
           </td>
           <td>
             <button id="${editButtonId}" type="button" class="btn btn-sm btn-secondary" title="Edit" data-bs-toggle="modal"
@@ -42,6 +42,9 @@ function loadConnectorSecrets() {
 
       $("#connector-secrets-table tbody:last-child").append(row);
 
+      $("#connector-secrets-table tbody:last-child tr:last-child .secretName").text(secret.name);
+      $("#connector-secrets-table tbody:last-child tr:last-child code").text(secret.value);
+
       $("#" + editButtonId).click(function () {
         // fill modal
         $("#edit-connector-secret-name").val(secret.name);
diff --git a/src/main/resources/public/js/view-process-instance.js b/src/main/resources/public/js/view-process-instance.js
index 7ff99220..a9f3048b 100644
--- a/src/main/resources/public/js/view-process-instance.js
+++ b/src/main/resources/public/js/view-process-instance.js
@@ -578,8 +578,6 @@ function loadVariablesOfProcessInstance() {
 
       let scopeElement = formatBpmnElementInstance(scope.element);
 
-      let valueFormatted = "<code>" + variable.value + "</code>";
-
       let lastUpdatedFormatted =
         '<div class="row row-cols-1">' +
         '<div class="col">' +
@@ -665,9 +663,7 @@ function loadVariablesOfProcessInstance() {
           "<td>" +
           variable.name +
           "</td>" +
-          "<td>" +
-          valueFormatted +
-          "</td>" +
+          "<td><code></code></td>" +
           "<td>" +
           scopeFormatted +
           "</td>" +
@@ -685,6 +681,8 @@ function loadVariablesOfProcessInstance() {
           "</td>" +
           "</tr>"
       );
+
+      $("#variables-of-process-instance-table > tbody:last-child > tr:last-child code").text(variable.value);
     });
   });
 }
@@ -719,8 +717,7 @@ function setVariablesModal() {
       const toastId = "set-variables-" + key;
       showNotificationSuccess(
         toastId,
-        "Variables set successfully",
-        "<code>" + variables + "</code>."
+        "Variables set successfully"
       );
 
       loadVariablesOfProcessInstance();
@@ -728,7 +725,7 @@ function setVariablesModal() {
     .fail(
       showFailure(
         "set-variables" + scope,
-        "Failed to set variables <code>" + variables + "</code>."
+        "Failed to set variables."
       )
     );
 }