close all bodies

cpraehaus · Mar 16, 2022 · dd09c5f · dd09c5f
1 parent 70964ec
commit dd09c5f
Show file tree

Hide file tree

Showing 9 changed files with 15 additions and 0 deletions.
diff --git a/provider/freemobile/freemobile.go b/provider/freemobile/freemobile.go
@@ -64,6 +64,7 @@ func (c *FreeMobile) Send(message sachet.Message) error {
 	if err != nil {
 		return err
 	}
+	defer response.Body.Close()
 
 	if response.StatusCode == http.StatusOK && err == nil {
 		return nil

diff --git a/provider/kannel/kannel.go b/provider/kannel/kannel.go
@@ -60,6 +60,7 @@ func (c *Kannel) Send(message sachet.Message) error {
 		if err != nil {
 			return err
 		}
+		defer response.Body.Close()
 
 		if response.StatusCode >= http.StatusBadRequest {
 			return fmt.Errorf("Failed sending sms. statusCode: %d", response.StatusCode)

diff --git a/provider/sap/sap.go b/provider/sap/sap.go
@@ -51,6 +51,7 @@ func (c *Sap) Send(message sachet.Message) error {
 	if err != nil {
 		return err
 	}
+	defer response.Body.Close()
 
 	if response.StatusCode == http.StatusOK && err == nil {
 		return nil

diff --git a/provider/sipgate/sipgate.go b/provider/sipgate/sipgate.go
@@ -65,6 +65,7 @@ func (c *Sipgate) Send(message sachet.Message) error {
 		if err != nil {
 			return err
 		}
+		defer response.Body.Close()
 
 		if response.StatusCode != http.StatusNoContent {
 			return fmt.Errorf("Failed sending sms. statusCode: %d", response.StatusCode)

diff --git a/provider/sms77/sms77.go b/provider/sms77/sms77.go
@@ -15,6 +15,8 @@ type Config struct {
 	Debug  bool   `yaml:"debug"`
 }
 
+var _ (sachet.Provider) = (*Sms77)(nil)
+
 // Sms77 contains the necessary values for the Sms77 provider.
 type Sms77 struct {
 	client *sms77api.Sms77API

diff --git a/provider/smsc/smsc.go b/provider/smsc/smsc.go
@@ -16,6 +16,8 @@ type Config struct {
 
 const SmscRequestTimeout = time.Second * 60
 
+var _ (sachet.Provider) = (*Smsc)(nil)
+
 type Smsc struct {
 	Login    string
 	Password string

diff --git a/provider/telegram/telegram.go b/provider/telegram/telegram.go
@@ -14,6 +14,8 @@ type Config struct {
 	DisableWebPagePreview bool   `yaml:"disable_web_page_preview"`
 }
 
+var _ (sachet.Provider) = (*Telegram)(nil)
+
 type Telegram struct {
 	bot    *tgbotapi.BotAPI
 	config *Config

diff --git a/provider/textmagic/textmagic.go b/provider/textmagic/textmagic.go
@@ -15,6 +15,8 @@ type Config struct {
 	APIKey   string `yaml:"api_key"`
 }
 
+var _ (sachet.Provider) = (*TextMagic)(nil)
+
 type TextMagic struct {
 	client *textmagic.APIClient
 	auth   context.Context

diff --git a/provider/turbosms/turbosms.go b/provider/turbosms/turbosms.go
@@ -19,6 +19,8 @@ type Config struct {
 	Apassword string `yaml:"password"`
 }
 
+var _ (sachet.Provider) = (*Turbosms)(nil)
+
 type Turbosms struct {
 	Login    string
 	Password string
@@ -99,6 +101,7 @@ func Request(c *http.Client, url string, payload []byte) ([]byte, error, int) {
 	if err != nil {
 		return nil, err, statuscode
 	}
+	defer resp.Body.Close()
 
 	body, err := ioutil.ReadAll(resp.Body)
 	if err != nil {