Skip to content

Conversation

spacewander
Copy link

No description provided.

pintsized and others added 9 commits March 16, 2021 12:06
* Avoid sending non-string body values

Fixes ledgetech#217

* Add newline at EOF

* Document support for table of strings in request body

* Revert change preventing non string request bodies

* Add tests for different request body types

* Ensure request body Content-Length is set correctly

* Don't blindly set request content length

* Add more explicit support for chunked request body iterators

* Use new transfer encoding utility

* Add tests for client body reader

Fails on chunked input until supported by ngx_lua

* Improve transfer encoding check

Co-authored-by: Thijs Schreijer <[email protected]>

Co-authored-by: Thijs Schreijer <[email protected]>
Also removed avoiding builds on doc only, since checks are required for
merging.
* Make flakey test a little less flakey

* Ensure Content-Length is dropped if Transfer-Encoding is specified

https://tools.ietf.org/html/rfc7230#section-3.3.3
* Bump version

* Don't ignore luarock spec files
* Ignore luarocks build artefacts

* Bump version to 1.16.1
* Add test to reproduce ledgetech#241

* Add error message for broken response status line

Fixes ledgetech#241

* Add test for an explicit error message string

See ledgetech#241
@CLAassistant
Copy link

CLAassistant commented Sep 1, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 5 committers have signed the CLA.

✅ spacewander
❌ bootjp
❌ Invizory
❌ pintsized
❌ Tieske
You have signed the CLA already but the status is still pending? Let us recheck it.

@spacewander spacewander marked this pull request as ready for review September 1, 2021 05:56
@spacewander
Copy link
Author

Surpass #7

@spacewander spacewander merged commit a0500ff into api7:master Sep 2, 2021
@spacewander spacewander deleted the ledgetech-master branch September 2, 2021 07:11
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.

6 participants