-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add POST requests #9
Comments
related: also, it only works with the data/text payload as request parameter. i haven't dug into the details but i had problems with large files. my quick solution was two-fold: commul/tint@836f7e4
i think, it would be nice to - in absence of a text= request parameter - expect the payload in the request body. about the Content-Length: maybe using it is unnecessary. this depends on the /exact/ behaviour of the InputStream... my understanding would be that waiting for '-1' aka EOF should be enough (and that's what is being done in your original implementation). but: with a long enough request the data might come in blocks, and whether the InputStream blocks until all data has been received or it only blocks until one block has been received (and it would be necessary to drain the buffer and request the rest...) i don't know - i wanted to be sure... |
What about this? 5053d51 |
Tint server does not work with POST requests.
The text was updated successfully, but these errors were encountered: