Merge pull request #8 from Brightscout/MI-380

Julien Neuhart · web-flow · commit 921a34c52087 · 2019-10-09T20:24:59.000+02:00
Check for error before writing new file in client.Store()
diff --git a/client.go b/client.go
@@ -94,6 +94,10 @@ func (c *Client) Store(req Request, dest string) error {
 		return err
 	}
 	defer resp.Body.Close()
+
+	if resp.StatusCode != http.StatusOK {
+		return errors.New("failed to generate the result PDF")
+	}
 	return writeNewFile(dest, resp.Body)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -94,6 +94,10 @@ func (c *Client) Store(req Request, dest string) error {`
`94`	`94`	`return err`
`95`	`95`	`}`
`96`	`96`	`defer resp.Body.Close()`
	`97`	`+`
	`98`	`+ if resp.StatusCode != http.StatusOK {`
	`99`	`+ return errors.New("failed to generate the result PDF")`
	`100`	`+ }`
`97`	`101`	`return writeNewFile(dest, resp.Body)`
`98`	`102`	`}`
`99`	`103`