feat: change route in service provider

dewanakl · Feb 25, 2024 · 55538e8 · 55538e8
1 parent 1af0425
commit 55538e8
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 9 deletions.
diff --git a/app/Kernel.php b/app/Kernel.php
@@ -59,7 +59,6 @@ public function middlewares(): array
             \App\Middleware\CorsMiddleware::class,
             \App\Middleware\XSSMiddleware::class,
             \App\Middleware\GzipMiddleware::class,
-            \App\Middleware\CsrfMiddleware::class
         ];
     }
 

diff --git a/app/Middleware/CorsMiddleware.php b/app/Middleware/CorsMiddleware.php
@@ -18,7 +18,7 @@ public function handle(Request $request, Closure $next)
         $header = respond()->getHeader();
         $header->set('Access-Control-Allow-Origin', '*');
 
-        $vary = (!$header->has('Vary')) ? [] : explode(', ', $header->get('Vary'));
+        $vary = $header->has('Vary') ? explode(', ', $header->get('Vary')) : [];
         $vary = array_unique([...$vary, 'Accept', 'Access-Control-Request-Method', 'Access-Control-Request-Headers', 'Origin', 'User-Agent']);
         $header->set('Vary', join(', ', $vary));
 
@@ -39,7 +39,7 @@ public function handle(Request $request, Closure $next)
 
         $header->set(
             'Access-Control-Allow-Headers',
-            $request->server->get('HTTP_ACCESS_CONTROL_REQUEST_HEADERS', 'Accept, Authorization, Content-Type, Origin, Token, User-Agent')
+            $request->server->get('HTTP_ACCESS_CONTROL_REQUEST_HEADERS', 'Accept, Authorization, Content-Type, Origin, User-Agent')
         );
 
         return respond()->setCode(Respond::HTTP_NO_CONTENT);

diff --git a/app/Middleware/GzipMiddleware.php b/app/Middleware/GzipMiddleware.php
@@ -28,19 +28,19 @@ public function handle(Request $request, Closure $next): Stream|Respond
             return $response;
         }
 
-        if (!str_contains($request->server->get('HTTP_ACCEPT_ENCODING'), 'gzip')) {
+        if (!str_contains($request->server->get('HTTP_ACCEPT_ENCODING', ''), 'gzip')) {
             return $response;
         }
 
-        $compressed = gzencode($response->getContent(false), 1);
+        $compressed = gzencode($response->getContent(false), 3);
 
         if ($compressed === false) {
             return $response;
         }
 
         $response->setContent($compressed);
 
-        $vary = (!$response->headers->has('Vary')) ? [] : explode(', ', $response->headers->get('Vary'));
+        $vary = $response->headers->has('Vary') ? explode(', ', $response->headers->get('Vary')) : [];
         $vary = array_unique([...$vary, 'Accept-Encoding']);
 
         $response->headers

diff --git a/app/Providers/RouteServiceProvider.php b/app/Providers/RouteServiceProvider.php
@@ -2,6 +2,7 @@
 
 namespace App\Providers;
 
+use App\Middleware\CsrfMiddleware;
 use Core\Facades\Provider;
 use Core\Routing\Route;
 use Core\Routing\Router;
@@ -16,6 +17,11 @@ class RouteServiceProvider extends Provider
     public function booting()
     {
         $this->app->singleton(Router::class);
-        Route::setRouteFromCacheIfExist();
+
+        if (!Route::setRouteFromCacheIfExist()) {
+            Route::middleware(CsrfMiddleware::class)->group(function () {
+                Route::setRouteFromFile();
+            });
+        }
     }
 }
diff --git a/composer.json b/composer.json
@@ -9,7 +9,7 @@
     "license": "MIT",
     "require": {
         "php": "^8.1",
-        "kamu/framework": "^3.2"
+        "kamu/framework": "^3.3"
     },
     "require-dev": {
         "fakerphp/faker": "^1.23",
@@ -35,4 +35,4 @@
     },
     "minimum-stability": "stable",
     "prefer-stable": true
-}
+}