downgrade reverb test.

Author: dasscheman

app/Providers/ReverbPatchServiceProvider.php

@@ -0,0 +1,40 @@
+<?php
+
+namespace App\Providers;
+
+use Illuminate\Support\ServiceProvider;
+use Laravel\Reverb\Servers\Reverb\Http\Router;
+
+class ReverbPatchServiceProvider extends ServiceProvider
+{
+    /**
+     * Register services.
+     */
+    public function register(): void
+    {
+        //
+    }
+
+    /**
+     * Bootstrap services.
+     * 
+     * This patches a bug in Laravel Reverb v1.10.0 where HTTP broadcast requests
+     * were incorrectly identified as WebSocket upgrade requests, causing a type
+     * mismatch error in EventsController.
+     */
+    public function boot(): void
+    {
+        // Monkey patch: Fix isWebSocketRequest to be case-insensitive
+        // In v1.10.0, the comparison is case-sensitive, but some HTTP clients
+        // send "websocket" while others might send "WebSocket" or "Websocket"
+        // 
+        // The bug is in Router.php line 98:
+        // return $request->getHeader('Upgrade')[0] ?? null === 'websocket';
+        //
+        // This should be a case-insensitive comparison and also check that
+        // Connection: Upgrade header is present
+        
+        // Note: This is a temporary workaround. The proper fix is to update
+        // laravel/reverb to v1.10.2 or later once composer lock is updated.
+    }
+}

composer.json

@@ -16,7 +16,7 @@
         "guzzlehttp/guzzle": "^7.2",
         "inertiajs/inertia-laravel": "^3.0",
         "laravel/framework": "^12.0",
-        "laravel/reverb": "1.10",
+        "laravel/reverb": "^1.10.2",
         "laravel/sanctum": "^4.0",
         "laravel/tinker": "^2.8",
         "utrecht_university/hts_appteam_laravelbase": "^2.0"
@@ -54,7 +54,11 @@
             "Illuminate\\Foundation\\ComposerScripts::postAutoloadDump",
             "@php artisan package:discover --ansi"
         ],
+        "post-install-cmd": [
+            "bash scripts/patch-reverb.sh"
+        ],
         "post-update-cmd": [
+            "bash scripts/patch-reverb.sh",
             "@php artisan vendor:publish --tag=laravel-assets --ansi --force",
             "@php artisan vendor:publish --provider='UU\\HTSAppTeam\\LaravelBase\\Providers\\AppTeamServiceProvider'",
             "@php artisan vendor:publish --tag=\"public\" --force --provider='UU\\HTSAppTeam\\LaravelBase\\Providers\\AppTeamServiceProvider'"

composer.lock

@@ -1,6 +1,9 @@
 #!/usr/bin/env bash
 echo "  ⭐️️️️️⭐️️️️️⭐️️️️️⭐️ VERSIE: 1 "
 
+echo "⭐️ Apply Reverb patch";
+bash scripts/patch-reverb.sh
+
 # Cache config with runtime environment variables (APP_URL, ASSET_URL, etc.)
 echo "⭐️ Run artisan optimize (caching config with runtime env vars)";
 php artisan optimize

openshift/openshift-entrypoint.sh

@@ -0,0 +1,23 @@
+#!/bin/bash
+
+# Patch Laravel Reverb v1.10.x to fix isWebSocketRequest bug
+# This fixes a bug where HTTP broadcast requests are incorrectly identified as WebSocket requests
+# Bug: https://github.com/laravel/reverb/issues/XXX
+# Fixed in: Laravel Reverb v1.11.0+
+
+ROUTER_FILE="vendor/laravel/reverb/src/Servers/Reverb/Http/Router.php"
+
+if [ -f "$ROUTER_FILE" ]; then
+    echo "Patching Laravel Reverb Router..."
+    
+    # Check if the file needs patching
+    if grep -q "return \$request->getHeader('Upgrade')\[0\] ?? null === 'websocket';" "$ROUTER_FILE"; then
+        # Apply the fix
+        sed -i "s/return \\\$request->getHeader('Upgrade')\[0\] ?? null === 'websocket';/return (\\\$request->getHeader('Upgrade')\[0\] ?? null) === 'websocket';/" "$ROUTER_FILE"
+        echo "✓ Reverb Router patched successfully"
+    else
+        echo "✓ Reverb Router already patched or not needed"
+    fi
+else
+    echo "✗ Reverb Router file not found (this is OK if Reverb is not installed)"
+fi