Merge pull request #104 from Poltuu/up

ADD : onbeforeAdd hook
Poltuu · Jun 17, 2020 · 6940357 · 6940357
2 parents 367f308 + b7c0533
commit 6940357
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/Harpoon.Registrations.EFStorage/WebHookRegistrationStore.cs b/Harpoon.Registrations.EFStorage/WebHookRegistrationStore.cs
@@ -117,6 +117,11 @@ public async Task<WebHookRegistrationStoreResult> InsertWebHookAsync(IPrincipal
                 Filters = webHook.Filters.Select(f => new WebHookFilter { Trigger = f.Trigger }).ToList()
             };
 
+            if (!OnBeforeAdd(dbWebHook))
+            {
+                return WebHookRegistrationStoreResult.Success;
+            }
+
             try
             {
                 _context.Add(dbWebHook);
@@ -130,6 +135,13 @@ public async Task<WebHookRegistrationStoreResult> InsertWebHookAsync(IPrincipal
             }
         }
 
+        /// <summary>
+        /// Returns a value indicating if the webhook should be added. Give the opportunity to modify the webhook as well.
+        /// </summary>
+        /// <param name="webhook"></param>
+        /// <returns></returns>
+        protected virtual bool OnBeforeAdd(WebHook webhook) => true;
+
         /// <inheritdoc />
         public async Task<WebHookRegistrationStoreResult> UpdateWebHookAsync(IPrincipal user, IWebHook webHook, CancellationToken cancellationToken = default)
         {