fix: Existing Hooks are replaced with new ones on restart (#33)

* fix: Hooks should attempt to update if already exists

* switch method to PUT

* delete and put when already exists

* delete then post

Author: cbaker6

Sources/App/entrypoint.swift

@@ -37,11 +37,6 @@ enum Entrypoint {
         let app = Application(env)
 
         defer {
-            Task {
-                // This may not delete all because it's async
-                // Be sure to delete manually in dashboard
-                await deleteHooks(app)
-            }
             app.shutdown()
         }
 

Sources/ParseServerSwift/Models/HookFunction.swift

@@ -59,6 +59,14 @@ extension HookFunction {
                 if error.containedIn([.webhookError]) && method == .POST {
                     // swiftlint:disable:next line_length
                     configuration.logger.warning("Hook Function: \"\(String(describing: hookFunction))\"; warning: \(error); on server: \(parseServerURLString)")
+                    try await Self.method(.DELETE,
+                                          path,
+                                          name: name,
+                                          parseServerURLStrings: parseServerURLStrings)
+                    return try await Self.method(method,
+                                                 path,
+                                                 name: name,
+                                                 parseServerURLStrings: parseServerURLStrings)
                 } else {
                     // swiftlint:disable:next line_length
                     configuration.logger.error("Could not \(method) Hook Function: \"\(String(describing: hookFunction))\"; error: \(error); on server: \(parseServerURLString)")

Sources/ParseServerSwift/Models/HookTrigger.swift

@@ -69,6 +69,16 @@ extension HookTrigger {
                 if error.containedIn([.webhookError]) && method == .POST {
                     // swiftlint:disable:next line_length
                     configuration.logger.warning("Hook Trigger: \"\(String(describing: hookTrigger))\"; warning: \(error); on server: \(parseServerURLString)")
+                    try await Self.method(.DELETE,
+                                          path,
+                                          className: className,
+                                          triggerName: triggerName,
+                                          parseServerURLStrings: parseServerURLStrings)
+                    return try await Self.method(method,
+                                                 path,
+                                                 className: className,
+                                                 triggerName: triggerName,
+                                                 parseServerURLStrings: parseServerURLStrings)
                 } else {
                     // swiftlint:disable:next line_length
                     configuration.logger.error("Could not \(method) Hook Trigger: \"\(String(describing: hookTrigger))\"; error: \(error); on server: \(parseServerURLString)")