Merge pull request #453 from sinamics/async-webhhook

Improve webhook to send http request async

Dockerfile

@@ -8,11 +8,11 @@ FROM base AS deps
 WORKDIR /app
 
 # Install Prisma Client - remove if not using Prisma
-RUN npx prisma generate
+COPY package.json yarn.lock* package-lock.json* pnpm-lock.yaml* ./
 COPY prisma ./
+RUN npx prisma generate
 
 # Install dependencies based on the preferred package manager
-COPY package.json yarn.lock* package-lock.json* pnpm-lock.yaml* ./
 RUN \
     if [ -f yarn.lock ]; then yarn --frozen-lockfile; \
     elif [ -f package-lock.json ]; then npm ci; \

src/utils/webhook.ts

@@ -11,21 +11,23 @@ export const sendWebhook = async <T extends HookBase>(data: T): Promise<void> =>
 
 	for (const webhook of webhookData) {
 		if ((webhook.eventTypes as string[]).includes(data.hookType)) {
-			try {
-				const response = await fetch(webhook.url, {
-					method: "POST",
-					headers: { "Content-Type": "application/json" },
-					body: JSON.stringify(data),
-				});
+			(async () => {
+				try {
+					const response = await fetch(webhook.url, {
+						method: "POST",
+						headers: { "Content-Type": "application/json" },
+						body: JSON.stringify(data),
+					});
 
-				if (!response.ok) {
-					throw new Error(
-						`Failed to send webhook: ${response.status} ${response.statusText}`,
-					);
+					if (!response.ok) {
+						console.error(
+							`Failed to send webhook: ${response.status} ${response.statusText}`,
+						);
+					}
+				} catch (error) {
+					console.error(`Error sending webhooks: ${error.message}`);
 				}
-			} catch (error) {
-				throw new Error(`Error sending webhook: ${error.message}`);
-			}
+			})();
 		}
 	}
 };