Merge pull request #105 from arnested/harden

Harden code

Author: arnested

.golangci.yml

@@ -1,10 +1,11 @@
 linters:
   enable-all: true
   disable:
+    - depguard
     - errname
     - exhaustivestruct
-    - nosnakecase
     - exhaustruct
+    - nosnakecase
     - testpackage
 
 linters-settings:

post.go

@@ -29,7 +29,7 @@ func (c *Client) do(ctx context.Context, form url.Values) ([]byte, error) {
 	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
 
 	resp, err := c.httpClient().Do(req)
-	if err != nil {
+	if (err != nil) || (resp == nil) {
 		return nil, fmt.Errorf("failed to contact API: %w", err)
 	}
 

post_test.go

@@ -135,7 +135,7 @@ func TestUpdateOK(t *testing.T) {
 	_, err := client.Update(ctx, records)
 	if err != nil {
 		var netErr net.Error
-		if errors.As(err, &netErr) && netErr.Timeout() {
+		if errors.As(err, &netErr) && (netErr != nil) && netErr.Timeout() {
 			t.Errorf("Timeoutsss: %s", netErr)
 		} else {
 			t.Errorf("Successful post should return OK but failed with error: %s", err)