From 61afad37fed3b9a64012ab2b8bdfb78a0db901b6 Mon Sep 17 00:00:00 2001
From: Bilfeldt <abilfeldt@gmail.com>
Date: Fri, 25 Jul 2025 10:15:00 +0200
Subject: [PATCH] Update LaravelHttpClientLoggerServiceProvider.php

Remove the default `null` parameter for the logWith macro as that would cause a failue since the LoggingMiddleware strictly require a logger
---
 src/LaravelHttpClientLoggerServiceProvider.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/LaravelHttpClientLoggerServiceProvider.php b/src/LaravelHttpClientLoggerServiceProvider.php
index 5d084e3..96371cd 100644
--- a/src/LaravelHttpClientLoggerServiceProvider.php
+++ b/src/LaravelHttpClientLoggerServiceProvider.php
@@ -59,7 +59,7 @@ public function packageBooted()
             }
         });
 
-        PendingRequest::macro('logWith', function (?HttpLoggerInterface $logger = null): PendingRequest {
+        PendingRequest::macro('logWith', function (HttpLoggerInterface $logger): PendingRequest {
             /** @var \Illuminate\Http\Client\PendingRequest $this */
             return $this->withMiddleware((new LoggingMiddleware($logger, new LogAllFilter()))->__invoke());
         });