From 8d905974beec1b981907d80d74b010fce7f176b6 Mon Sep 17 00:00:00 2001
From: Eli Bosley <ekbosley@gmail.com>
Date: Tue, 28 Jan 2025 15:00:19 -0500
Subject: [PATCH] Update api/src/utils.ts

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
---
 api/src/utils.ts | 23 ++++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/api/src/utils.ts b/api/src/utils.ts
index 293e0c293..ff9c32402 100644
--- a/api/src/utils.ts
+++ b/api/src/utils.ts
@@ -245,6 +245,9 @@ export function handleAuthError(
     throw new UnauthorizedException(`${operation}: ${errorMessage}`);
 }
 
+import { access, constants } from 'node:fs/promises';
+import { dirname } from 'node:path';
+
 /**
  * Helper method to allow backing up a single file to a .bak file.
  * @param path the file to backup, creates a .bak file in the same directory
@@ -252,11 +255,29 @@ export function handleAuthError(
  */
 export const backupFile = async (path: string, throwOnMissing = true): Promise<void> => {
     try {
+        // Validate path
+        if (!path) {
+            throw new Error('File path cannot be empty');
+        }
+        
+        // Check if source file exists and is readable
+        await access(path, constants.R_OK);
+        
+        // Check if backup directory is writable
+        await access(dirname(path), constants.W_OK);
+        
         const backupPath = path + '.bak';
         await copyFile(path, backupPath);
     } catch (err) {
+        const error = err as NodeJS.ErrnoException;
         if (throwOnMissing) {
-            throw new Error(`File does not exist: ${path}`);
+            throw new Error(
+                error.code === 'ENOENT'
+                    ? `File does not exist: ${path}`
+                    : error.code === 'EACCES'
+                    ? `Permission denied: ${path}`
+                    : `Failed to backup file: ${error.message}`
+            );
         }
     }
 };