alihan/fn-serverless
1
0
Fork 0
mirror of https://github.com/fnproject/fn.git synced 2022-10-28 21:29:17 +03:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #335 from fnproject/sanitize-cl

Browse Source 
fix neg content length
This commit is contained in:
Reed Allman
2017-09-18 13:58:12 -07:00
committed by GitHub
parent 0280df0cdf 0f50e4b7fc
commit b95aed5ecf
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
api/server/runner.go
View File

@@ -69,7 +69,11 @@ func (s *Server) serve(c *gin.Context, appName, path string) {
if model.Type == "async" { if model.Type == "async" {
// TODO we should push this into GetCall somehow (CallOpt maybe) or maybe agent.Queue(Call) ? // TODO we should push this into GetCall somehow (CallOpt maybe) or maybe agent.Queue(Call) ?
buf := bytes.NewBuffer(make([]byte, c.Request.ContentLength)[:0]) // TODO sync.Pool me contentLength := c.Request.ContentLength
if contentLength < 128 { // contentLength could be -1 or really small, sanitize
contentLength = 128
}
buf := bytes.NewBuffer(make([]byte, int(contentLength))[:0]) // TODO sync.Pool me
_, err := buf.ReadFrom(c.Request.Body) _, err := buf.ReadFrom(c.Request.Body)
if err != nil { if err != nil {
handleErrorResponse(c, models.ErrInvalidPayload) handleErrorResponse(c, models.ErrInvalidPayload)