Fixed forgeapi http transport to support proxy settings from envvars.…

… Made forgeapi error logging more useful. Updated README with proxy settings.

README.md

@@ -23,6 +23,7 @@ Pufctl allows you to quickly add modules to a Puppetfile from a git source or th
     * [Working With Puppetfiles](#working-with-puppetfiles)
     * [Git and Authentication](#git-and-authentication)
     * [Using a Config File](#using-a-config-file)
+    * [Proxy Settings](#proxy-settings)
 * [Features](#features)
 * [Usage](#usage)
     * [Basic Examples](#basic-examples)
@@ -88,6 +89,13 @@ Nearly every option provided by a command line flag can be configured using a co
 
 The default location for the config file is `$HOME/.pufctl.yaml`. You can use a config file at a different path by passing the path to the `--config` global flag.
 
+### Proxy Settings
+
+If you are behind a proxy, so can configure Pufctl to use the proxy by settings the following environment variables:
+
+* `HTTP_PROXY`
+* `HTTPS_PROXY`
+* `USE_PROXIES` (optional)
 
 ## Features
 

pkg/forgeapi/errors.go

@@ -20,7 +20,7 @@ type GetError struct {
 }
 
 func (r *GetError) Error() string {
-	return fmt.Sprintf("%s HTTP GET request failed. URL: %s, Headers: %#v, Error: %#v", prefix(), r.URL, r.Headers, r.Err)
+	return fmt.Sprintf("%s HTTP GET request failed. URL: %s, Headers: %#v, Error: %s", prefix(), r.URL, r.Headers, r.Err.Error())
 }
 
 // GetNon200Error provides an error wrapper for when
@@ -41,7 +41,7 @@ type JSONDecodeError struct {
 }
 
 func (r *JSONDecodeError) Error() string {
-	return fmt.Sprintf("%s Failed to decode HTTP response body: %#v", prefix(), r.Err)
+	return fmt.Sprintf("%s Failed to decode HTTP response body: %#v", prefix(), r.Err.Error())
 }
 
 // FetchError provides a wrapper for errors encountered
@@ -51,7 +51,7 @@ type FetchError struct {
 }
 
 func (r *FetchError) Error() string {
-	return fmt.Sprintf("%s Fetch failed: %#v", prefix(), r.Err)
+	return fmt.Sprintf("%s Fetch failed: %#v", prefix(), r.Err.Error())
 }
 
 // ListError provides a wrapper for errors encountered
@@ -61,5 +61,5 @@ type ListError struct {
 }
 
 func (r *ListError) Error() string {
-	return fmt.Sprintf("%s List failed: %#v", prefix(), r.Err)
+	return fmt.Sprintf("%s List failed: %#v", prefix(), r.Err.Error())
 }

pkg/forgeapi/forgeapi.go

@@ -20,6 +20,7 @@ var (
 		MaxIdleConns:       3,
 		IdleConnTimeout:    5 * time.Second,
 		DisableCompression: true,
+		Proxy:              http.ProxyFromEnvironment,
 	}
 )