Skip to content

Fixed #10 #11

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixed #10 #11

wants to merge 1 commit into from

Conversation

theroch
Copy link

@theroch theroch commented Mar 8, 2021

Check if $http_response_header is set before using it.
This fixes #10

@waleedfarooq123
Copy link

Check if $http_response_header is set before using it. This fixes #10

Hi, will you please merge this branch into master

@tenzap
Copy link

tenzap commented Feb 7, 2025

The null coalescing operator (??) is new in PHP 7.0.
If one wants to support versions prior to it, like 5.6, maybe one should use a more compatible syntax.

Having issue #10 fixed would be nice though.

@theroch
Copy link
Author

theroch commented Feb 10, 2025

The minimum requirement for this lib is PHP 7.0.
Consideration of 5.6 and below is not necessary.

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.

Bug: Undefined variable $http_response_header in Client->send()
3 participants