fixed input stream in mode serializing

mslaga commented 2 years ago

ContentLength is optional in the HTTP header and cannot specify how much data will be copied to process. https://golang.org/src/net/http/request.go?s=12527:12599 // The value -1 indicates that the length is unknown.

Also, not all data will be available for io.LimitReader they could arrive later.

copy stderr to logs - it will be helpful for troubleshooting with function

Description

Copy the req.InputReader stream to stdin in loop until EOF is reached.

Motivation and Context

Not works on slow machines like raspberry pi

How Has This Been Tested?

Not works at all on raspberry pi

Types of changes

[ x] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x ] I've read the CONTRIBUTION guide
[x ] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[x ] All new and existing tests passed.

derek[bot] commented 2 years ago

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide. Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

alexellis commented 2 years ago

Hi thanks for your interest, but you will need to raise an issue with a unit test or failing scenario that is reproducible before a pull request can be considered.

Do fill out the whole template and don't delete any of the fields.

You'll find out more on how to contribute in the Contributing guide in the faas repo.

Best,

Alex

derek[bot] commented 2 years ago

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide. Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

openfaas / of-watchdog