lyft / presto-gateway

A load balancer / proxy / gateway for prestodb
Apache License 2.0
358 stars 156 forks source link

FIX: Fix body length changed #46

Closed hamlet-lee closed 5 years ago

hamlet-lee commented 5 years ago

In current implementation, when downstream posts a content with "\r\n", the body length is changed.
Which makes request proxied to upstream has "Content-Length" > body length
So upstream server will wait forever (until timeout) for the request.

I have tested this pull request.

puneetjaiswal commented 5 years ago

Awesome debugging - https://github.com/hamlet-lee/blog/blob/master/2019-05-22.md 👍

hamlet-lee commented 5 years ago

@puneetjaiswal my pleasure :)