search

joshrps / laravel-shopify-API-wrapper

Interface designed for Shopify apps created with Laravel
MIT License
93 stars 48 forks source link

Add Content-Length header #34

Closed amandasaffer closed 6 years ago

amandasaffer commented 6 years ago

To prevent Shopify from throwing 411 Length Required error.

amandasaffer commented 6 years ago

@joshrps Please accept this? Issue discussed here: https://github.com/joshrps/laravel-shopify-API-wrapper/issues/33

eliranm commented 6 years ago

@joshrps Can you accept this issue? It do solve the reported problem.

xewl commented 6 years ago

src/RocketCode/Shopify/API.php has yet to contain a default header of Content-Length: 0. This mostly occurs on empty POST requests, like on: order/[order_id]/fulfilments/[fulfilment_id]/complete.json