Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pusher doesn't respect REVERB_SERVER_PATH value #321

Closed
SaeedDev94 opened this issue Mar 15, 2025 · 1 comment
Closed

Pusher doesn't respect REVERB_SERVER_PATH value #321

SaeedDev94 opened this issue Mar 15, 2025 · 1 comment

Comments

@SaeedDev94
Copy link

Reverb Version

1.4.7

Laravel Version

12.2.0

PHP Version

8.4.5

Description

I like to add a path via REVERB_SERVER_PATH
I added the path to both reverb.servers.reverb.path and broadcasting.connections.reverb.options.path
But pusher tries to send requests to a url without the prefix (path)
This is because Pusher doesn't take the REVERB_SERVER_PATH as the prefix for its settings#base_path
I created a PR for this pusher/pusher-http-php#399 but it seems the repo is not active as I saw a similar PR which was created a long time ago! pusher/pusher-http-php#388
So maybe the laravel team should fork pusher/pusher-http-php and edit it for the framework ??
This is very important as the request path also affects on Controller#verifySignature() method and causes HttpException (Authentication signature invalid.)

Steps To Reproduce

  • set REVERB_SERVER_PATH for both reverb.servers.reverb.path and broadcasting.connections.reverb.options.path
  • start reverb: php artisan reverb:start --debug
  • start queue: php artisan queue:listen
  • start tinker: php artisan tinker
  • send a test broadcast: \Illuminate\Support\Facades\Broadcast::on('test')->via('reverb')->with(['id' => 6])->send();
  • you can see error log that says can't send the post request (because Pusher ignored the REVERB_SERVER_PATH)
@SaeedDev94
Copy link
Author

My bad!! I have to set broadcasting.connections.reverb.options.base_path
BTW documentations didn't mention about this!
Just saw it in the PR !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant