Refactor error handling to improve clarity and maintainability

server/notification-providers/threema.js

@@ -41,41 +41,35 @@ class Threema extends NotificationProvider {
             await axios.post(url, new URLSearchParams(data), config);
             return "Threema notification sent successfully.";
         } catch (error) {
-            this.handleApiError(error);
+            const errorMessage = this.handleApiError(error);
-            this.throwGeneralAxiosError(error);
+            this.throwGeneralAxiosError(errorMessage);
         }
     }
 
     /**
      * Handle Threema API errors
      * @param {any} error The error to handle
-     * @returns {void}
+     * @returns {string} Additional error context
      */
     handleApiError(error) {
-        if (error.response) {
+        if (!error.response) {
-            const status = error.response.status;
+            return error.message;
-            switch (status) {
+        }
-                case 400:
+        switch (error.response.status) {
-                    error.message = "Invalid recipient identity or account not set up for basic mode (400).";
+            case 400:
-                    break;
+                return "Invalid recipient identity or account not set up for basic mode (400).";
-                case 401:
+            case 401:
-                    error.message = "Incorrect API identity or secret (401).";
+                return "Incorrect API identity or secret (401).";
-                    break;
+            case 402:
-                case 402:
+                return "No credits remaining (402).";
-                    error.message = "No credits remaining (402).";
+            case 404:
-                    break;
+                return "Recipient not found (404).";
-                case 404:
+            case 413:
-                    error.message = "Recipient not found (404).";
+                return "Message is too long (413).";
-                    break;
+            case 500:
-                case 413:
+                return "Temporary internal server error (500).";
-                    error.message = "Message is too long (413).";
+            default:
-                    break;
+                return error.message;
-                case 500:
-                    error.message = "Temporary internal server error (500).";
-                    break;
-                default:
-                    break;
-            }
         }
     }
 }