published configs won't have "web" middleware added (#36)

.github/workflows/pullpreview.yml

@@ -35,6 +35,7 @@ jobs:
         uses: actions/upload-artifact@v4
         with:
           name: transmorpher-env
+          include-hidden-files: true
           path: |
             placeholderSoFilePathsArePreserved
             amigor/.env.transmorpher

CHANGELOG.md

@@ -0,0 +1,11 @@
+# Release Notes
+
+## [v0.4.0](https://github.com/cybex-gmbh/laravel-transmorpher-client/compare/v0.3.0...v0.4.0)
+
+> [!WARNING]
+> Breaking change!
+
+- Route middlewares are now fully configurable
+  - the `SubstituteBindings` middleware is now always applied as it is necessary to resolve route model bindings
+  - when `transmorpher.routeMiddleware` is not set, the `web` and `auth` middlewares are applied
+  - already published config files have to be adjusted accordingly

amigor/config/transmorpher.php

@@ -3,10 +3,10 @@
 return [
     'client_name' => env('TRANSMORPHER_CLIENT_NAME'),
 
-    // The middleware applied to routes provided by this package. The 'web' middleware is necessary and applied by default.
-    'routeMiddleware' => [
-//        'auth'
-    ],
+    // The middlewares applied to routes provided by this package:
+    // - the "SubstituteBindings" middleware will be applied additionally.
+    // - "web" and "auth" middlewares will be applied when this is not set.
+    'routeMiddleware' => [],
 
     'api' => [
         // Optionally, specify the Transmorpher API version which should be used. For supported versions, check the SupportedApiVersion enum.

config/transmorpher.php

@@ -3,10 +3,10 @@
 return [
     'client_name' => env('TRANSMORPHER_CLIENT_NAME'),
 
-    // The middleware applied to routes provided by this package. The 'web' middleware is necessary and applied by default.
-    'routeMiddleware' => [
-        'auth'
-    ],
+    // The middlewares applied to routes provided by this package:
+    // - the "SubstituteBindings" middleware will be applied additionally.
+    // - "web" and "auth" middlewares will be applied when this is not set.
+//    'routeMiddleware' => ['web', 'auth'],
 
     'api' => [
         // Optionally, specify the Transmorpher API version which should be used. For supported versions, check the SupportedApiVersion enum.

src/TransmorpherServiceProvider.php

@@ -2,6 +2,7 @@
 
 namespace Transmorpher;
 
+use Illuminate\Routing\Middleware\SubstituteBindings;
 use Illuminate\Support\Facades\Blade;
 use Illuminate\Support\Facades\Route;
 use Illuminate\Support\ServiceProvider;
@@ -62,7 +63,7 @@ public function register(): void
     protected function registerRoutes(): void
     {
         Route::post(config('transmorpher.api.notifications_route'), ApiController::class)->name('transmorpherNotifications');
-        Route::middleware(array_merge(['web'], config('transmorpher.routeMiddleware', [])))->group(function () {
+        Route::middleware(array_merge(config('transmorpher.routeMiddleware', ['web', 'auth']), [SubstituteBindings::class]))->group(function () {
             Route::post('transmorpher/{transmorpherMedia}/token', [UploadController::class, 'getUploadToken'])->name('transmorpherUploadToken');
             Route::post('transmorpher/handleUploadResponse/{transmorpherUpload}', [UploadController::class, 'handleUploadResponse'])->name('transmorpherHandleUploadResponse');
             Route::post('transmorpher/{transmorpherMedia}/state', [UploadStateController::class, 'getState'])->name('transmorpherState');