Hide response headers per default; Add flag -i for showing headers (#5

)

* Flag -i/--include now triggers inclusion of HTTP headers in output

The default is now to not print the headers. This complies with the
default behaviour of curl.

pkg/cmd/http.go

@@ -18,6 +18,7 @@ import (
 func NewHTTPCommand() *cobra.Command {
 	var method, data string
 	var headers []string
+	var includeHeaders bool
 
 	cmd := &cobra.Command{
 		Use:   "http <path>",
@@ -61,23 +62,26 @@ func NewHTTPCommand() *cobra.Command {
 			}
 			defer resp.Body.Close()
 
-			return dumpResponse(resp)
+			return dumpResponse(resp, includeHeaders)
 		},
 	}
 
+	cmd.Flags().BoolVarP(&includeHeaders, "include", "i", false, "Include HTTP headers in the output")
 	cmd.Flags().StringVarP(&data, "data", "d", "", "HTTP POST data")
 	cmd.Flags().StringVarP(&method, "request", "X", "GET", "Specify request command to use")
 	cmd.Flags().StringSliceVarP(&headers, "header", "H", nil, "Pass custom header(s) to server")
 
 	return cmd
 }
 
-func dumpResponse(resp *http.Response) error {
-	respDump, err := httputil.DumpResponse(resp, false)
-	if err != nil {
-		return err
+func dumpResponse(resp *http.Response, includeHeaders bool) error {
+	if includeHeaders {
+		respDump, err := httputil.DumpResponse(resp, false)
+		if err != nil {
+			return err
+		}
+		fmt.Println(string(respDump))
 	}
-	fmt.Println(string(respDump))
-	_, err = io.Copy(os.Stdout, resp.Body)
+	_, err := io.Copy(os.Stdout, resp.Body)
 	return err
 }