Improved error handling of connection errors

twikey · Jan 2, 2024 · ff7e2dd · ff7e2dd
1 parent 4ccfe4e
commit ff7e2dd
Show file tree

Hide file tree

Showing 5 changed files with 44 additions and 15 deletions.
diff --git a/document.go b/document.go
@@ -323,7 +323,10 @@ func (c *Client) DocumentFeed(
 		req.Header.Set("Accept", "application/json")
 		req.Header.Set("User-Agent", c.UserAgent)
 
-		res, _ := c.HTTPClient.Do(req)
+		res, err := c.HTTPClient.Do(req)
+		if err != nil {
+			return err
+		}
 		if res.StatusCode == 200 {
 			payload, _ := ioutil.ReadAll(res.Body)
 			var updates MandateUpdates
@@ -365,7 +368,10 @@ func (c *Client) DownloadPdf(ctx context.Context, mndtId string, downloadFile st
 	req.Header.Add("Authorization", c.apiToken)
 
 	absPath, _ := filepath.Abs(downloadFile)
-	res, _ := c.HTTPClient.Do(req)
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return err
+	}
 	if res.StatusCode == 200 {
 		payload, _ := ioutil.ReadAll(res.Body)
 
@@ -403,7 +409,10 @@ func (c *Client) DocumentDetail(ctx context.Context, mndtId string, force bool)
 	req.Header.Set("User-Agent", c.UserAgent)
 	req.Header.Add("Authorization", c.apiToken)
 
-	res, _ := c.HTTPClient.Do(req)
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return nil, err
+	}
 	if res.StatusCode == 200 {
 		payload, _ := ioutil.ReadAll(res.Body)
 

diff --git a/invoice.go b/invoice.go
@@ -199,7 +199,10 @@ func (c *Client) InvoiceAdd(ctx context.Context, invoiceRequest *NewInvoiceReque
 		}
 	}
 
-	res, _ := c.HTTPClient.Do(req)
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return nil, err
+	}
 	if res.StatusCode == 200 {
 		payload, _ := ioutil.ReadAll(res.Body)
 		c.Debug.Println("TwikeyInvoice: ", string(payload))
@@ -280,7 +283,10 @@ func (c *Client) InvoiceDetail(ctx context.Context, invoiceIdOrNumber string, si
 	req.Header.Set("User-Agent", c.UserAgent)
 	req.Header.Add("Authorization", c.apiToken)
 
-	res, _ := c.HTTPClient.Do(req)
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return nil, err
+	}
 	if res.StatusCode == 200 {
 		payload, _ := ioutil.ReadAll(res.Body)
 
@@ -327,7 +333,10 @@ func (c *Client) InvoiceAction(ctx context.Context, invoiceIdOrNumber string, ac
 	req.Header.Set("User-Agent", c.UserAgent)
 	req.Header.Add("Authorization", c.apiToken)
 
-	res, _ := c.HTTPClient.Do(req)
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return err
+	}
 	if res.StatusCode == 204 {
 		return nil
 	}
@@ -354,7 +363,10 @@ func (c *Client) InvoicePayment(ctx context.Context, invoiceIdOrNumber string, m
 	req.Header.Set("User-Agent", c.UserAgent)
 	req.Header.Add("Authorization", c.apiToken)
 
-	res, _ := c.HTTPClient.Do(req)
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return err
+	}
 	if res.StatusCode == 204 {
 		return nil
 	}

diff --git a/paylink.go b/paylink.go
@@ -119,8 +119,10 @@ func (c *Client) PaylinkFeed(ctx context.Context, callback func(paylink *Paylink
 		req.Header.Add("Authorization", c.apiToken)
 		req.Header.Set("User-Agent", c.UserAgent)
 
-		res, _ := c.HTTPClient.Do(req)
-
+		res, err := c.HTTPClient.Do(req)
+		if err != nil {
+			return err
+		}
 		if res.StatusCode == 200 {
 			payload, _ := ioutil.ReadAll(res.Body)
 			_ = res.Body.Close()

diff --git a/session.go b/session.go
@@ -98,8 +98,10 @@ func (c *Client) logout() {
 	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
 	req.Header.Set("Authorization", c.apiToken)
 
-	res, _ := c.HTTPClient.Do(req)
-	if res.StatusCode != 200 {
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		c.Debug.Println("Error in logout from Twikey:", err)
+	} else if res.StatusCode != 200 {
 		c.Debug.Println("Error in logout from Twikey:", res.StatusCode)
 	}
 }
diff --git a/transaction.go b/transaction.go
@@ -148,8 +148,10 @@ func (c *Client) TransactionFeed(ctx context.Context, callback func(transaction
 		req.Header.Add("Authorization", c.apiToken)
 		req.Header.Set("User-Agent", c.UserAgent)
 
-		res, _ := c.HTTPClient.Do(req)
-
+		res, err := c.HTTPClient.Do(req)
+		if err != nil {
+			return err
+		}
 		if res.StatusCode == 200 {
 			payload, _ := ioutil.ReadAll(res.Body)
 			_ = res.Body.Close()
@@ -193,8 +195,10 @@ func (c *Client) TransactionCollect(ctx context.Context, template string, prenot
 	req.Header.Add("Content-Type", "application/x-www-form-urlencoded")
 	req.Header.Add("Authorization", c.apiToken)
 	req.Header.Set("User-Agent", c.UserAgent)
-	res, _ := c.HTTPClient.Do(req)
-
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return "", err
+	}
 	if res.StatusCode == 200 {
 		payload, _ := ioutil.ReadAll(res.Body)
 		_ = res.Body.Close()