Merge pull request #1756 from elementary-data/ele-3826-fix-teams-webhook-alerts

MikaKerman · web-flow · commit 6a8a49158ec4 · 2024-11-19T14:13:36.000+02:00
Response object has no attribute body, replaced with text
diff --git a/elementary/clients/slack/client.py b/elementary/clients/slack/client.py
@@ -243,7 +243,7 @@ def send_message(self, message: SlackMessageSchema, **kwargs) -> bool:
 
         else:
             logger.error(
-                f"Could not post message to slack via webhook - {self.webhook}. Error: {response.body}"
+                f"Could not post message to slack via webhook - {self.webhook}. Error: {response.text}"
             )
             return False
 
diff --git a/elementary/clients/teams/client.py b/elementary/clients/teams/client.py
@@ -75,7 +75,7 @@ def send_message(self, **kwargs) -> bool:
             logger.error(
                 "Could not post message to teams via webhook - %s. Error: %s",
                 {self.webhook},
-                {response.body},
+                {response.text},
             )
         return False
 

Original file line number	Diff line number	Diff line change
`@@ -243,7 +243,7 @@ def send_message(self, message: SlackMessageSchema, **kwargs) -> bool:`
`243`	`243`
`244`	`244`	`else:`
`245`	`245`	`logger.error(`
`246`		`- f"Could not post message to slack via webhook - {self.webhook}. Error: {response.body}"`
	`246`	`+ f"Could not post message to slack via webhook - {self.webhook}. Error: {response.text}"`
`247`	`247`	`)`
`248`	`248`	`return False`
`249`	`249`
Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ def send_message(self, **kwargs) -> bool:`
`75`	`75`	`logger.error(`
`76`	`76`	`"Could not post message to teams via webhook - %s. Error: %s",`
`77`	`77`	`{self.webhook},`
`78`		`- {response.body},`
	`78`	`+ {response.text},`
`79`	`79`	`)`
`80`	`80`	`return False`
`81`	`81`