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

Fix Backward Compatibility for servers running Pusher < 5.0.0 #324

Merged
merged 2 commits into from
Jun 2, 2022

Conversation

Attia-Ahmed
Copy link
Contributor

Description

Closes #322, calling get_object_vars($result->channels) function in Pusher.php on response of old versions of pusher servers <v5.0.0 lead to TypeError Exception, as an array is passed instead of object.

CHANGELOG

  • [FIXED] get_object_vars() error on /src/Pusher.php

Copy link

@Stejsi Stejsi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine, merge that into master soon pleas.

@sonologico sonologico merged commit 8abfae4 into pusher:master Jun 2, 2022
@Attia-Ahmed Attia-Ahmed deleted the fix_backward_compatibility branch June 2, 2022 14:56
@Delia-SimpleKYC
Copy link

@Attia-Ahmed could this fix be included in branch "7.0.2" ? I need this version to be working because of this issue: beyondcode/laravel-websockets#1041

Thanks

@Delia-SimpleKYC
Copy link

@Stejsi I have noticed that you have placed a thumbs-up on my request :) Could you tell me when the update will be done? We need this to be solved as soon as possible, otherwise we will have to configure another server..... Thanks in advance!!

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

Successfully merging this pull request may close these issues.

get_object_vars() error on /src/Pusher.php:459
4 participants