Making sure webhook is signed, and payload is sent

JustSteveKing · May 31, 2023 · c9156f9 · c9156f9
1 parent 0c2e671
commit c9156f9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/Builder/PendingWebhook.php b/src/Builder/PendingWebhook.php
@@ -142,7 +142,9 @@ public function send(Method $method = Method::POST): Response
             /** @phpstan-ignore-next-line  */
             $this->request->withHeaders(
                 headers: [
-                    strval(config('webhooks.signing.header')) => $this->signature ?: $this->signed()->signature,
+                    strval(config('webhooks.signing.header')) => $this->signer->sign(
+                        payload: $this->payload,
+                    ),
                 ],
             );
         }
@@ -151,6 +153,9 @@ public function send(Method $method = Method::POST): Response
         return $this->request->send(
             method: $method->value,
             url: $this->url,
+            options: [
+                'json' => $this->payload,
+            ]
         )->throw();
     }
 }
diff --git a/src/Providers/PackageServiceProvider.php b/src/Providers/PackageServiceProvider.php
@@ -37,6 +37,7 @@ public function register(): void
                 signer: $this->app->make(
                     abstract: SigningContract::class,
                 ),
+                signature: strval(config('webhooks.signing.key')),
             ),
         );
     }