Merge pull request #510 from opcodesio/l13-support

Add Laravel 13 support

Author: arukompas

.github/workflows/run-tests.yml

@@ -13,8 +13,8 @@ jobs:
       fail-fast: true
       matrix:
         os: [ubuntu-latest, windows-latest]
-        php: [8.2, 8.1, 8.0]
-        laravel: [9.*, 10.*, 11.*, 12.*]
+        php: [8.4, 8.3, 8.2, 8.1, 8.0]
+        laravel: [9.*, 10.*, 11.*, 12.*, 13.*]
         stability: [prefer-lowest, prefer-stable]
         exclude:
           - php: 8.0
@@ -23,11 +23,21 @@ jobs:
             laravel: 11.*
           - php: 8.0
             laravel: 12.*
+          - php: 8.0
+            laravel: 13.*
           - php: 8.1
             laravel: 11.*
           - php: 8.1
             laravel: 12.*
+          - php: 8.1
+            laravel: 13.*
+          - php: 8.2
+            laravel: 9.*
           - php: 8.2
+            laravel: 13.*
+          - php: 8.3
+            laravel: 9.*
+          - php: 8.4
             laravel: 9.*
         include:
           - laravel: 9.*
@@ -46,6 +56,10 @@ jobs:
             testbench: 10.*
             pest: 3.*
             collision: 8.*
+          - laravel: 13.*
+            testbench: 11.*
+            pest: 4.*
+            collision: 8.*
 
     name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.stability }} - ${{ matrix.os }}
 
@@ -71,4 +85,4 @@ jobs:
           composer update --${{ matrix.stability }} --prefer-dist --no-interaction
 
       - name: Execute tests
-        run: vendor/bin/pest
+        run: vendor/bin/pest --no-coverage

composer.json

@@ -20,17 +20,17 @@
     ],
     "require": {
         "php": "^8.0",
-        "illuminate/contracts": "^8.0|^9.0|^10.0|^11.0|^12.0",
+        "illuminate/contracts": "^8.0|^9.0|^10.0|^11.0|^12.0|^13.0",
         "opcodesio/mail-parser": "^0.2.3"
     },
     "require-dev": {
         "guzzlehttp/guzzle": "^7.2",
         "itsgoingd/clockwork": "^5.1",
         "laravel/pint": "^1.0",
         "nunomaduro/collision": "^7.0|^8.0",
-        "orchestra/testbench": "^7.6|^8.0|^9.5|^10.0",
-        "pestphp/pest": "^2.0|^3.7",
-        "pestphp/pest-plugin-laravel": "^2.0|^3.1",
+        "orchestra/testbench": "^7.6|^8.0|^9.5|^10.0|^11.0",
+        "pestphp/pest": "^2.0|^3.7|^4.4",
+        "pestphp/pest-plugin-laravel": "^2.0|^3.1|^4.1",
         "spatie/test-time": "^1.3"
     },
     "suggest": {

config/log-viewer.php

@@ -3,6 +3,8 @@
 use Opcodes\LogViewer\Enums\SortingMethod;
 use Opcodes\LogViewer\Enums\SortingOrder;
 use Opcodes\LogViewer\Enums\Theme;
+use Opcodes\LogViewer\Http\Middleware\AuthorizeLogViewer;
+use Opcodes\LogViewer\Http\Middleware\EnsureFrontendRequestsAreStateful;
 
 return [
 
@@ -102,7 +104,7 @@
 
     'middleware' => [
         'web',
-        \Opcodes\LogViewer\Http\Middleware\AuthorizeLogViewer::class,
+        AuthorizeLogViewer::class,
     ],
 
     /*
@@ -115,8 +117,8 @@
     */
 
     'api_middleware' => [
-        \Opcodes\LogViewer\Http\Middleware\EnsureFrontendRequestsAreStateful::class,
-        \Opcodes\LogViewer\Http\Middleware\AuthorizeLogViewer::class,
+        EnsureFrontendRequestsAreStateful::class,
+        AuthorizeLogViewer::class,
     ],
 
     'api_stateful_domains' => env('LOG_VIEWER_API_STATEFUL_DOMAINS') ? explode(',', env('LOG_VIEWER_API_STATEFUL_DOMAINS')) : null,

src/Facades/Cache.php

@@ -2,12 +2,13 @@
 
 namespace Opcodes\LogViewer\Facades;
 
+use Illuminate\Cache\Repository;
 use Illuminate\Support\Facades\Facade;
 
 /**
  * @mixin \Illuminate\Contracts\Cache\Repository
  *
- * @see \Illuminate\Cache\Repository
+ * @see Repository
  */
 class Cache extends Facade
 {

src/Facades/LogViewer.php

@@ -10,10 +10,11 @@
 use Opcodes\LogViewer\LogFileCollection;
 use Opcodes\LogViewer\LogFolder;
 use Opcodes\LogViewer\LogFolderCollection;
+use Opcodes\LogViewer\LogViewerService;
 use Opcodes\LogViewer\Readers\LogReaderInterface;
 
 /**
- * @see \Opcodes\LogViewer\LogViewerService
+ * @see LogViewerService
  *
  * @method static string version()
  * @method static string timezone()

src/Http/Middleware/EnsureFrontendRequestsAreStateful.php

@@ -2,7 +2,13 @@
 
 namespace Opcodes\LogViewer\Http\Middleware;
 
+use Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse;
+use Illuminate\Cookie\Middleware\EncryptCookies;
+use Illuminate\Foundation\Http\Middleware\VerifyCsrfToken;
+use Illuminate\Http\Request;
+use Illuminate\Http\Response;
 use Illuminate\Routing\Pipeline;
+use Illuminate\Session\Middleware\StartSession;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Str;
 
@@ -16,9 +22,9 @@ class EnsureFrontendRequestsAreStateful
     /**
      * Handle the incoming requests.
      *
-     * @param  \Illuminate\Http\Request  $request
+     * @param  Request  $request
      * @param  callable  $next
-     * @return \Illuminate\Http\Response
+     * @return Response
      */
     public function handle($request, $next)
     {
@@ -30,10 +36,10 @@ function ($request, $next) {
 
                 return $next($request);
             },
-            static::resolveMiddleware('sanctum.middleware.encrypt_cookies', \Illuminate\Cookie\Middleware\EncryptCookies::class),
-            \Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse::class,
-            \Illuminate\Session\Middleware\StartSession::class,
-            static::resolveMiddleware('sanctum.middleware.verify_csrf_token', \Illuminate\Foundation\Http\Middleware\VerifyCsrfToken::class),
+            static::resolveMiddleware('sanctum.middleware.encrypt_cookies', EncryptCookies::class),
+            AddQueuedCookiesToResponse::class,
+            StartSession::class,
+            static::resolveMiddleware('sanctum.middleware.verify_csrf_token', VerifyCsrfToken::class),
         ] : [])->then(function ($request) use ($next) {
             return $next($request);
         });
@@ -55,7 +61,7 @@ protected function configureSecureCookieSessions()
     /**
      * Determine if the given request is from the first-party application frontend.
      *
-     * @param  \Illuminate\Http\Request  $request
+     * @param  Request  $request
      * @return bool
      */
     public static function fromFrontend($request)
@@ -91,7 +97,7 @@ public static function fromFrontend($request)
     /**
      * Check if the referer/origin domain matches the current request's domain.
      *
-     * @param  \Illuminate\Http\Request  $request
+     * @param  Request  $request
      * @param  string  $refererDomain
      * @return bool
      */

src/Http/Resources/LogResource.php

@@ -3,9 +3,10 @@
 namespace Opcodes\LogViewer\Http\Resources;
 
 use Illuminate\Http\Resources\Json\JsonResource;
+use Opcodes\LogViewer\Logs\Log;
 
 /**
- * @mixin \Opcodes\LogViewer\Logs\Log
+ * @mixin Log
  */
 class LogResource extends JsonResource
 {

src/LogViewerService.php

@@ -3,6 +3,7 @@
 namespace Opcodes\LogViewer;
 
 use Composer\InstalledVersions;
+use GuzzleHttp\Client;
 use Illuminate\Auth\Access\AuthorizationException;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\File;
@@ -202,7 +203,7 @@ public function getFolder(?string $folderIdentifier): ?LogFolder
 
     public function supportsHostsFeature(): bool
     {
-        return class_exists(\GuzzleHttp\Client::class);
+        return class_exists(Client::class);
     }
 
     public function resolveHostsUsing(callable $callback): void

src/Logs/Log.php

@@ -28,7 +28,7 @@ class Log
     /** @var string The regular expression group key, which contains the message */
     public static string $regexMessageKey = 'message';
 
-    /** @var array|\string[][] The columns displayed on the frontend, and which data they should display */
+    /** @var array|string[][] The columns displayed on the frontend, and which data they should display */
     public static array $columns = [
         ['label' => 'Datetime', 'data_path' => 'datetime'],
         ['label' => 'Severity', 'data_path' => 'level'],

tests/Feature/Authorization/CanViewLogViewerTest.php

@@ -1,7 +1,9 @@
 <?php
 
+use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Gate;
 use Opcodes\LogViewer\Facades\LogViewer;
+use Opcodes\LogViewer\LogViewerServiceProvider;
 
 use function Pest\Laravel\get;
 use function Pest\Laravel\getJson;
@@ -20,7 +22,7 @@
 
 test('the "auth" callback is given with a Request object to check against', function () {
     LogViewer::auth(function ($request) {
-        expect($request)->toBeInstanceOf(\Illuminate\Http\Request::class);
+        expect($request)->toBeInstanceOf(Request::class);
 
         return true;
     });
@@ -61,7 +63,7 @@
     app()->detectEnvironment(fn () => 'production');
     expect(app()->isProduction())->toBeTrue();
     // need to reload the routes in order for the new middleware to take place.
-    (new \Opcodes\LogViewer\LogViewerServiceProvider(app()))->boot();
+    (new LogViewerServiceProvider(app()))->boot();
 
     get(route('log-viewer.index'))->assertOk();
 });