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

Add support for formatting large request bodies #22

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

adduc
Copy link

@adduc adduc commented Mar 2, 2018

PHP's escapeshellarg throws a fatal error on bad input (null bytes, strings that are too long). To workaround this, use symfony/process to escape arguments instead.

Version 3 of symfony/process was used instead of 4 because of Cuzzle's support for PHP 5.5

Solves #20

LeTucTuc
LeTucTuc previously approved these changes May 6, 2020
cholladay0816
cholladay0816 previously approved these changes Mar 10, 2021
composer.json Outdated
@@ -4,7 +4,8 @@
"require": {
"php": ">=5.5.0",
"guzzlehttp/guzzle": "^6.0",
"psr/log": "^1.0"
"psr/log": "^1.0",
"symfony/process": "^3.0"

Choose a reason for hiding this comment

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

Use ^3.0|^4.0|^5.0 otherwise it cannot be installed on newer symfony versions

matheustiny
matheustiny previously approved these changes Oct 1, 2021
@adduc adduc dismissed stale reviews from matheustiny, cholladay0816, and LeTucTuc via a9b35a8 November 10, 2021 06:38
@bigfoot90
Copy link

This repo seems to be unmaintained now.
This same task was accomplished by a forked repository https://github.com/octoper/cuzzle/pull/1/files
@adduc Please open a new MR on october/cuzzle repo, and post. this same fix.

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.

5 participants