Use http.DefaultClient if no timeout option is specified

Author: Montana Flynn

src/targets/go/native.js

@@ -57,12 +57,14 @@ module.exports = function (source, options) {
   code.push('func main() {\n')
 
   // Create client
+  var client
   if (opts.timeout > 0) {
+    client = 'client'
     code.push('\tclient := http.Client{')
     code.push(util.format('\t\tTimeout: time.Duration(%s * time.Second),', opts.timeout))
     code.push('\t}\n')
   } else {
-    code.push('\tclient := &http.Client{}\n')
+    client = 'http.DefaultClient'
   }
 
   code.push(util.format('\turl := "%s"\n', source.fullUrl))
@@ -86,7 +88,7 @@ module.exports = function (source, options) {
   }
 
   // Make request
-  code.push(util.format('\tres, %s := client.Do(req)', errorPlaceholder))
+  code.push(util.format('\tres, %s := %s.Do(req)', errorPlaceholder, client))
   errorCheck()
 
   // Get Body

test/fixtures/output/go/native/application-form-encoded.go

@@ -9,8 +9,6 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har"
 
 	payload := strings.NewReader("foo=bar&hello=world")
@@ -19,7 +17,7 @@ func main() {
 
 	req.Header.Add("content-type", "application/x-www-form-urlencoded")
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)

test/fixtures/output/go/native/application-json.go

@@ -9,8 +9,6 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har"
 
 	payload := strings.NewReader("{\"number\": 1, \"string\": \"f\\\"oo\", \"arr\": [1, 2, 3], \"nested\": {\"a\": \"b\"}, \"arr_mix\": [1, \"a\", {\"arr_mix_nested\": {}}] }")
@@ -19,7 +17,7 @@ func main() {
 
 	req.Header.Add("content-type", "application/json")
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)

test/fixtures/output/go/native/cookies.go

@@ -8,15 +8,13 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har"
 
 	req, _ := http.NewRequest("POST", url, nil)
 
 	req.Header.Add("cookie", "foo=bar; bar=baz")
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)

test/fixtures/output/go/native/full.go

@@ -9,8 +9,6 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har?foo=bar&foo=baz&baz=abc&key=value"
 
 	payload := strings.NewReader("foo=bar")
@@ -21,7 +19,7 @@ func main() {
 	req.Header.Add("accept", "application/json")
 	req.Header.Add("content-type", "application/x-www-form-urlencoded")
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)

test/fixtures/output/go/native/headers.go

@@ -8,16 +8,14 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har"
 
 	req, _ := http.NewRequest("GET", url, nil)
 
 	req.Header.Add("accept", "application/json")
 	req.Header.Add("x-foo", "Bar")
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)

test/fixtures/output/go/native/multipart-data.go

@@ -9,8 +9,6 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har"
 
 	payload := strings.NewReader("-----011000010111000001101001\r\nContent-Disposition: form-data; name=\"foo\"; filename=\"hello.txt\"\r\nContent-Type: text/plain\r\n\r\nHello World\r\n-----011000010111000001101001--")
@@ -19,7 +17,7 @@ func main() {
 
 	req.Header.Add("content-type", "multipart/form-data; boundary=---011000010111000001101001")
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)

test/fixtures/output/go/native/multipart-file.go

@@ -9,8 +9,6 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har"
 
 	payload := strings.NewReader("-----011000010111000001101001\r\nContent-Disposition: form-data; name=\"foo\"; filename=\"hello.txt\"\r\nContent-Type: text/plain\r\n\r\n\r\n-----011000010111000001101001--")
@@ -19,7 +17,7 @@ func main() {
 
 	req.Header.Add("content-type", "multipart/form-data; boundary=---011000010111000001101001")
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)

test/fixtures/output/go/native/multipart-form-data.go

@@ -9,8 +9,6 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har"
 
 	payload := strings.NewReader("-----011000010111000001101001\r\nContent-Disposition: form-data; name=\"foo\"\r\n\r\nbar\r\n-----011000010111000001101001--")
@@ -19,7 +17,7 @@ func main() {
 
 	req.Header.Add("content-type", "multipart/form-data; boundary=---011000010111000001101001")
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)

test/fixtures/output/go/native/query.go

@@ -8,13 +8,11 @@ import (
 
 func main() {
 
-	client := &http.Client{}
-
 	url := "http://mockbin.com/har?foo=bar&foo=baz&baz=abc&key=value"
 
 	req, _ := http.NewRequest("GET", url, nil)
 
-	res, _ := client.Do(req)
+	res, _ := http.DefaultClient.Do(req)
 
 	defer res.Body.Close()
 	body, _ := ioutil.ReadAll(res.Body)