@hed-hog/core 0.0.390 → 0.0.394

package/src/webhook-integration/webhook-integration.service.ts

@@ -679,9 +679,63 @@ export class WebhookIntegrationService {
       req,
     };
 
-    return method === 'GET'
-      ? adapter.verify(request)
-      : adapter.handle(request);
+    // Sem isto, webhook de protocolo nao deixa rastro nenhum: o adaptador se
+    // autentica sozinho e devolve 403 mudo, entao uma integracao configurada
+    // errado fica indistinguivel de provedor que nunca chamou. A aba de Logs
+    // existe justamente para responder essa pergunta.
+    const startedAt = Date.now();
+    let result: WebhookProtocolResponse;
+
+    try {
+      result =
+        method === 'GET' ? await adapter.verify(request) : await adapter.handle(request);
+    } catch (error) {
+      await this.safeCreateCallLog({
+        integration,
+        uuid,
+        status: 'failed',
+        remoteIp: request.remoteIp,
+        durationMs: Date.now() - startedAt,
+        payload: body,
+        error,
+      });
+      throw error;
+    }
+
+    const failed = result.status >= 400;
+    await this.safeCreateCallLog({
+      integration,
+      uuid,
+      status: failed ? 'failed' : 'success',
+      remoteIp: request.remoteIp,
+      durationMs: Date.now() - startedAt,
+      payload: body,
+      response: { status: result.status, protocol: integration.protocol },
+      error: failed
+        ? (result.logReason ?? `Protocol adapter returned ${result.status}.`)
+        : undefined,
+    });
+
+    return result;
+  }
+
+  /**
+   * O log e observabilidade: falhar ao grava-lo nao pode derrubar a entrega, ou
+   * um problema de escrita viraria retentativa infinita no provedor.
+   */
+  private async safeCreateCallLog(
+    input: Parameters<WebhookIntegrationService['createCallLog']>[0],
+  ) {
+    try {
+      return await this.createCallLog(input);
+    } catch (error) {
+      this.logger.error(
+        `Could not record webhook call log for ${input.uuid}: ${
+          (error as Error)?.message
+        }`,
+      );
+      return null;
+    }
   }
 
   async executePublic(uuid: string, body: any, headers: any, req: Request) {

package/src/webhook-integration/webhook-protocol.registry.ts

@@ -12,6 +12,12 @@ export type WebhookProtocolResponse = {
   status: number;
   contentType?: string;
   body: string | object;
+  /**
+   * Motivo da recusa, para o log de chamadas. Nunca vai na resposta HTTP: um
+   * webhook publico que explica por que recusou ajuda quem esta atacando tanto
+   * quanto quem esta configurando.
+   */
+  logReason?: string;
 };
 
 export type WebhookProtocolRequest = {