@oneuptime/common 10.0.52 → 10.0.54

package/Server/Utils/Workspace/Slack/Actions/ScheduledMaintenance.ts

@@ -24,7 +24,7 @@ import { LIMIT_PER_PROJECT } from "../../../../../Types/Database/LimitMax";
 import { DropdownOption } from "../../../../../UI/Components/Dropdown/Dropdown";
 import ScheduledMaintenanceState from "../../../../../Models/DatabaseModels/ScheduledMaintenanceState";
 import ScheduledMaintenanceStateService from "../../../../Services/ScheduledMaintenanceStateService";
-import logger from "../../../Logger";
+import logger, { LogAttributes } from "../../../Logger";
 import SortOrder from "../../../../../Types/BaseDatabase/SortOrder";
 import Monitor from "../../../../../Models/DatabaseModels/Monitor";
 import MonitorService from "../../../../Services/MonitorService";
@@ -622,8 +622,14 @@ export default class SlackScheduledMaintenanceActions {
             isRoot: true,
           });
         } catch (err) {
-          logger.error("Error logging button interaction:");
-          logger.error(err);
+          logger.error("Error logging button interaction:", {
+            projectId: slackRequest.projectId?.toString(),
+            scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+          } as LogAttributes);
+          logger.error(err, {
+            projectId: slackRequest.projectId?.toString(),
+            scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+          } as LogAttributes);
           // Don't throw the error, just log it so the main flow continues
         }
       }
@@ -753,8 +759,14 @@ export default class SlackScheduledMaintenanceActions {
             isRoot: true,
           });
         } catch (err) {
-          logger.error("Error logging button interaction:");
-          logger.error(err);
+          logger.error("Error logging button interaction:", {
+            projectId: slackRequest.projectId?.toString(),
+            scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+          } as LogAttributes);
+          logger.error(err, {
+            projectId: slackRequest.projectId?.toString(),
+            scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+          } as LogAttributes);
           // Don't throw the error, just log it so the main flow continues
         }
       }
@@ -805,8 +817,12 @@ export default class SlackScheduledMaintenanceActions {
         },
       });
 
-    logger.debug("Scheduled Maintenance States: ");
-    logger.debug(scheduledMaintenanceStates);
+    logger.debug("Scheduled Maintenance States: ", {
+      projectId: data.slackRequest.projectId?.toString(),
+    } as LogAttributes);
+    logger.debug(scheduledMaintenanceStates, {
+      projectId: data.slackRequest.projectId?.toString(),
+    } as LogAttributes);
 
     const dropdownOptions: Array<DropdownOption> = scheduledMaintenanceStates
       .map((state: ScheduledMaintenanceState) => {
@@ -819,8 +835,12 @@ export default class SlackScheduledMaintenanceActions {
         return option.label !== "" || option.value !== "";
       });
 
-    logger.debug("Dropdown Options: ");
-    logger.debug(dropdownOptions);
+    logger.debug("Dropdown Options: ", {
+      projectId: data.slackRequest.projectId?.toString(),
+    } as LogAttributes);
+    logger.debug(dropdownOptions, {
+      projectId: data.slackRequest.projectId?.toString(),
+    } as LogAttributes);
 
     const statePickerDropdown: WorkspaceDropdownBlock = {
       _type: "WorkspaceDropdownBlock",
@@ -1133,8 +1153,9 @@ export default class SlackScheduledMaintenanceActions {
   }): Promise<void> {
     logger.debug(
       "Handling emoji reaction for Scheduled Maintenance with data:",
+      { channelId: data.channelId } as LogAttributes,
     );
-    logger.debug(data);
+    logger.debug(data, { channelId: data.channelId } as LogAttributes);
 
     const { teamId, reaction, userId, channelId, messageTs } = data;
 
@@ -1145,6 +1166,7 @@ export default class SlackScheduledMaintenanceActions {
     if (!isPrivateNoteEmoji && !isPublicNoteEmoji) {
       logger.debug(
         `Emoji "${reaction}" is not a supported note emoji. Ignoring.`,
+        { channelId: data.channelId } as LogAttributes,
       );
       return;
     }
@@ -1167,6 +1189,7 @@ export default class SlackScheduledMaintenanceActions {
     if (!projectAuth || !projectAuth.projectId || !projectAuth.authToken) {
       logger.debug(
         "No project auth found for team ID. Ignoring emoji reaction.",
+        { channelId: data.channelId } as LogAttributes,
       );
       return;
     }
@@ -1193,6 +1216,10 @@ export default class SlackScheduledMaintenanceActions {
     if (!workspaceLog || !workspaceLog.scheduledMaintenanceId) {
       logger.debug(
         "No scheduled maintenance found linked to this channel. Ignoring emoji reaction.",
+        {
+          projectId: projectId?.toString(),
+          channelId: channelId,
+        } as LogAttributes,
       );
       return;
     }
@@ -1227,6 +1254,10 @@ export default class SlackScheduledMaintenanceActions {
     if (!userAuth || !userAuth.userId) {
       logger.debug(
         "No OneUptime user found for Slack user. Ignoring emoji reaction.",
+        {
+          projectId: projectId?.toString(),
+          scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+        } as LogAttributes,
       );
       return;
     }
@@ -1242,13 +1273,22 @@ export default class SlackScheduledMaintenanceActions {
         messageTs: messageTs,
       });
     } catch (err) {
-      logger.error("Error fetching message text:");
-      logger.error(err);
+      logger.error("Error fetching message text:", {
+        projectId: projectId?.toString(),
+        scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+      } as LogAttributes);
+      logger.error(err, {
+        projectId: projectId?.toString(),
+        scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+      } as LogAttributes);
       return;
     }
 
     if (!messageText) {
-      logger.debug("No message text found. Ignoring emoji reaction.");
+      logger.debug("No message text found. Ignoring emoji reaction.", {
+        projectId: projectId?.toString(),
+        scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+      } as LogAttributes);
       return;
     }
 
@@ -1273,6 +1313,10 @@ export default class SlackScheduledMaintenanceActions {
         if (hasExistingNote) {
           logger.debug(
             "Private note from this Slack message already exists. Skipping duplicate.",
+            {
+              projectId: projectId?.toString(),
+              scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+            } as LogAttributes,
           );
           return;
         }
@@ -1284,7 +1328,10 @@ export default class SlackScheduledMaintenanceActions {
           userId: oneUptimeUserId,
           postedFromSlackMessageId: postedFromSlackMessageId,
         });
-        logger.debug("Private note added successfully.");
+        logger.debug("Private note added successfully.", {
+          projectId: projectId?.toString(),
+          scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+        } as LogAttributes);
       } else if (isPublicNoteEmoji) {
         noteType = "public";
 
@@ -1298,6 +1345,10 @@ export default class SlackScheduledMaintenanceActions {
         if (hasExistingNote) {
           logger.debug(
             "Public note from this Slack message already exists. Skipping duplicate.",
+            {
+              projectId: projectId?.toString(),
+              scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+            } as LogAttributes,
           );
           return;
         }
@@ -1309,13 +1360,22 @@ export default class SlackScheduledMaintenanceActions {
           userId: oneUptimeUserId,
           postedFromSlackMessageId: postedFromSlackMessageId,
         });
-        logger.debug("Public note added successfully.");
+        logger.debug("Public note added successfully.", {
+          projectId: projectId?.toString(),
+          scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+        } as LogAttributes);
       } else {
         return;
       }
     } catch (err) {
-      logger.error("Error saving note:");
-      logger.error(err);
+      logger.error("Error saving note:", {
+        projectId: projectId?.toString(),
+        scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+      } as LogAttributes);
+      logger.error(err, {
+        projectId: projectId?.toString(),
+        scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+      } as LogAttributes);
       return;
     }
 
@@ -1340,10 +1400,19 @@ export default class SlackScheduledMaintenanceActions {
         text: confirmationMessage,
       });
 
-      logger.debug("Confirmation message sent successfully.");
+      logger.debug("Confirmation message sent successfully.", {
+        projectId: projectId?.toString(),
+        scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+      } as LogAttributes);
     } catch (err) {
-      logger.error("Error sending confirmation message:");
-      logger.error(err);
+      logger.error("Error sending confirmation message:", {
+        projectId: projectId?.toString(),
+        scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+      } as LogAttributes);
+      logger.error(err, {
+        projectId: projectId?.toString(),
+        scheduledMaintenanceId: scheduledMaintenanceId?.toString(),
+      } as LogAttributes);
       // Don't throw - note was saved successfully, confirmation is best effort
     }
   }