[7.x] Prevent authentication route registrations when auth: false (#…

…289) * [7.x] Prevent authentication route registrations when `auth: false` Signed-off-by: Mior Muhammad Zaki <[email protected]> * wip Signed-off-by: Mior Muhammad Zaki <[email protected]> --------- Signed-off-by: Mior Muhammad Zaki <[email protected]>
orchestral · Jan 7, 2025 · 7131084 · 7131084
1 parent ebc16de
commit 7131084
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/Concerns/InteractsWithWorkbench.php b/src/Concerns/InteractsWithWorkbench.php
@@ -73,7 +73,7 @@ protected function getPackageProvidersUsingWorkbench($app): ?array
         $hasAuthentication = $config?->getWorkbenchAttributes()['auth'] ?? false;
         $providers = $config?->getExtraAttributes()['providers'] ?? [];
 
-        if ($hasAuthentication && class_exists('Orchestra\Workbench\AuthServiceProvider')) {
+        if ($hasAuthentication === true && class_exists('Orchestra\Workbench\AuthServiceProvider')) {
             $providers[] = 'Orchestra\Workbench\AuthServiceProvider';
         }
 

diff --git a/src/Workbench/Workbench.php b/src/Workbench/Workbench.php
@@ -86,10 +86,12 @@ public static function start(ApplicationContract $app, ConfigContract $config, a
      */
     public static function startWithProviders(ApplicationContract $app, ConfigContract $config): void
     {
-        static::start($app, $config, [
-            'Orchestra\Workbench\AuthServiceProvider',
+        $hasAuthentication = $config->getWorkbenchAttributes()['auth'] ?? false;
+
+        static::start($app, $config, array_filter([
+            $hasAuthentication === true && class_exists('Orchestra\Workbench\AuthServiceProvider') ? 'Orchestra\Workbench\AuthServiceProvider' : null,
             'Orchestra\Workbench\WorkbenchServiceProvider',
-        ]);
+        ]));
     }
 
     /**