Merge pull request #185 from fastly/joeshaw/http-body-no-stream

don't fire off goroutine if underlying body is *fastly.HTTPBody

Author: cee-dub

fsthttp/request.go

@@ -404,7 +404,7 @@ func (req *Request) Send(ctx context.Context, backend string) (*Response, error)
 	)
 
 	switch underlyingReaderFrom(req.Body).(type) {
-	case nil, *bytes.Buffer, *bytes.Reader, *strings.Reader:
+	case nil, *bytes.Buffer, *bytes.Reader, *strings.Reader, *fastly.HTTPBody:
 		streaming = false
 	}
 

integration_tests/request_upstream/main_test.go

@@ -113,7 +113,7 @@ func TestRequestUpstreamBody(t *testing.T) {
 		{name: "bytes.Buffer", body: bytes.NewBuffer(body), size: bodySize},
 		{name: "strings.Reader", body: strings.NewReader(string(body)), size: bodySize},
 		{name: "io.NopCloser", body: io.NopCloser(bytes.NewReader(body)), chunked: true},
-		{name: "fastly.HTTPBody", body: b, chunked: true},
+		{name: "fastly.HTTPBody", body: b, size: bodySize},
 	}
 
 	for _, tc := range testcases {