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

Handle CURLOPT_BINARYTRANSFER deprecation for newer PHP versions #54

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

Conversation

alissonlinneker
Copy link

Introduced a private property _use_binarytransfer to conditionally apply the CURLOPT_BINARYTRANSFER option only on PHP versions older than 5.4 (where it is not deprecated).

For newer PHP versions (5.4 and above), this option will no longer be set, ensuring compatibility and avoiding deprecated behavior.

The changes aim to maintain backward compatibility with older PHP environments, while ensuring a clean and warning-free behavior on newer ones.

Introduced a private property `_use_binarytransfer` to conditionally apply
the `CURLOPT_BINARYTRANSFER` option only on PHP versions older than 5.4
(where it is not deprecated).

For newer PHP versions (5.4 and above), this option will no longer be set,
ensuring compatibility and avoiding deprecated behavior.

The changes aim to maintain backward compatibility with older PHP
environments, while ensuring a clean and warning-free behavior on newer
ones.
@stefangabos
Copy link
Owner

i'll look over your changes but unfortunately you changed the formatting of the entire file so it's hard to see at a glance. but i'll look into it
thanks!

@stefangabos
Copy link
Owner

Just so it remains for posterity, it looks like this is the announcement

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.

2 participants