Fixes method post - Githubissues

andreif / uwsgi-tools

curl and proxy for remote uwsgi app server

BSD 3-Clause "New" or "Revised" License

43 stars 11 forks source link

Fixes method post #9

Open tomdottom opened 6 years ago

tomdottom commented 6 years ago

Adds CONTENT_LENGTH & CONTENT_TYPE so that post requests work correctly

Many frameworks/applications will rely on CONTENT_LENGTH to determine the length of the body appended to the end of uwsgi_packet_header + uwsgi_vars.

tomdottom commented 6 years ago

I should also point out that this work is base on top of #8. If you're happy with this PR you can merge it and simply close #8.

coveralls commented 6 years ago

Coverage remained the same at ?% when pulling f5faed61445d16b0cc7d962bdc0863eb18453bef on tomdottom:fixes-method-post into a86fea969c456153c9c4ed4983c76446887e7750 on andreif:master.

andreif commented 6 years ago

So now when #8 is merged, could you please rebase on master?

tomdottom commented 6 years ago

Rebased and ready for review.

tomdottom commented 6 years ago

Hey @andreif :wave: . Is there anything I left I need to do to get this PR merged?

andreif commented 6 years ago

@tomdottom Hey! Thanks a lot for all the commits! I just need to find few calm minutes to go through them 😄