From fda77fe46417732ceade52b82b8eb9799da9841e Mon Sep 17 00:00:00 2001
From: Daniel Hougaard <daniel@infisical.com>
Date: Thu, 3 Apr 2025 08:23:12 +0400
Subject: [PATCH] fix: better error handling & renamed handler function

---
 .../components/CmekVerifyModal.tsx            | 23 +++++++++++++------
 1 file changed, 16 insertions(+), 7 deletions(-)

diff --git a/frontend/src/pages/kms/OverviewPage/components/CmekVerifyModal.tsx b/frontend/src/pages/kms/OverviewPage/components/CmekVerifyModal.tsx
index db4857522..e678f328f 100644
--- a/frontend/src/pages/kms/OverviewPage/components/CmekVerifyModal.tsx
+++ b/frontend/src/pages/kms/OverviewPage/components/CmekVerifyModal.tsx
@@ -68,13 +68,22 @@ const VerifyForm = ({ cmek }: FormProps) => {
     }
   });
 
-  const handleSignData = async (formData: FormData) => {
+  const handleVerifyData = async (formData: FormData) => {
     try {
-      await cmekVerify.mutateAsync({ ...formData, keyId: cmek.id });
-      createNotification({
-        text: "Successfully signed data",
-        type: "success"
-      });
+      const result = await cmekVerify.mutateAsync({ ...formData, keyId: cmek.id });
+
+      if (result.signatureValid) {
+        createNotification({
+          text: "Successfully verified signature",
+          type: "success"
+        });
+      } else {
+        createNotification({
+          title: "Signature Verification Failed",
+          text: "The signature is invalid. The signature was not created using the same signing algorithm and key as the one used to sign the data. The data and signature may have been tampered with.",
+          type: "error"
+        });
+      }
     } catch (err) {
       console.error(err);
       createNotification({
@@ -98,7 +107,7 @@ const VerifyForm = ({ cmek }: FormProps) => {
   );
 
   return (
-    <form onSubmit={handleSubmit(handleSignData)}>
+    <form onSubmit={handleSubmit(handleVerifyData)}>
       {signatureValid !== undefined ? (
         <div className="mb-6 flex flex-col gap-2">
           <div className="flex items-center justify-between space-x-2">