Skip to content
This repository was archived by the owner on Sep 8, 2021. It is now read-only.

Post fix #7

Closed
wants to merge 6 commits into from
Closed

Post fix #7

wants to merge 6 commits into from

Conversation

atrope
Copy link
Contributor

@atrope atrope commented Dec 3, 2018

Hi,
Testing in our environment we found a bug when sending the content-length Header via ALB.
The curl call stucks which i think it's a bug: https://stackoverflow.com/questions/9340734/php-curl-data-not-posted-if-headers-are-set

I Managed to fix by making the header on-the fly with the actual sent body.
For this i changed the order where we call the CURLOPT_HTTPHEADER to after the $body variable is defined.

@atrope atrope closed this Dec 3, 2018
@atrope atrope deleted the postFix branch December 3, 2018 13:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant