added a Body.Close() to prevent memory leak

dstpierre · dstpierre · commit 7713da105848 · 2022-08-27T08:24:05.000-04:00
diff --git a/function/runtime.go b/function/runtime.go
@@ -181,6 +181,8 @@ func (env *ExecutionEnvironment) addHelpers(vm *goja.Runtime) {
 			if err != nil {
 				return vm.ToValue(Result{OK: false, Content: fmt.Sprintf("error calling fetch(): %s", err.Error())})
 			}
+			response.Body.Close()
+
 			return vm.ToValue(Result{OK: true, Content: HTTPResponse{Status: response.StatusCode, Body: string(bodyBytes)}})
 		}
 		return goja.Undefined()

Original file line number	Diff line number	Diff line change
`@@ -181,6 +181,8 @@ func (env ExecutionEnvironment) addHelpers(vm goja.Runtime) {`
`181`	`181`	`if err != nil {`
`182`	`182`	`return vm.ToValue(Result{OK: false, Content: fmt.Sprintf("error calling fetch(): %s", err.Error())})`
`183`	`183`	`}`
	`184`	`+ response.Body.Close()`
	`185`	`+`
`184`	`186`	`return vm.ToValue(Result{OK: true, Content: HTTPResponse{Status: response.StatusCode, Body: string(bodyBytes)}})`
`185`	`187`	`}`
`186`	`188`	`return goja.Undefined()`