Migrate GitHub abd GitLab integrations to new request system #350

Signed-off-by: tdruez <[email protected]>

Author: tdruez

workflow/integrations/github.py

@@ -71,13 +71,7 @@ def create_issue(self, repo_id, title, body="", labels=None):
         if labels:
             data["labels"] = labels
 
-        response = self.session.post(
-            url,
-            json=data,
-            timeout=self.default_timeout,
-        )
-        response.raise_for_status()
-        return response.json()
+        return self.post(url, json=data)
 
     def update_issue(self, repo_id, issue_id, title=None, body=None, state=None, labels=None):
         """Update an existing GitHub issue."""
@@ -92,26 +86,14 @@ def update_issue(self, repo_id, issue_id, title=None, body=None, state=None, lab
         if labels:
             data["labels"] = labels
 
-        response = self.session.patch(
-            url,
-            json=data,
-            timeout=self.default_timeout,
-        )
-        response.raise_for_status()
-        return response.json()
+        return self.patch(url, json=data)
 
     def post_comment(self, repo_id, issue_id, comment_body):
         """Post a comment on an existing GitHub issue."""
         url = f"{self.api_url}/repos/{repo_id}/issues/{issue_id}/comments"
         data = {"body": comment_body}
 
-        response = self.session.post(
-            url,
-            json=data,
-            timeout=self.default_timeout,
-        )
-        response.raise_for_status()
-        return response.json()
+        return self.post(url, json=data)
 
     @staticmethod
     def extract_github_repo_path(url):

workflow/integrations/gitlab.py

@@ -76,13 +76,7 @@ def create_issue(self, repo_id, title, body="", labels=None):
             # GitLab expects a comma-separated string for labels
             data["labels"] = ",".join(labels)
 
-        response = self.session.post(
-            url,
-            json=data,
-            timeout=self.default_timeout,
-        )
-        response.raise_for_status()
-        return response.json()
+        return self.post(url, json=data)
 
     def update_issue(self, repo_id, issue_id, title=None, body=None, state_event=None, labels=None):
         """Update an existing GitLab issue."""
@@ -99,27 +93,15 @@ def update_issue(self, repo_id, issue_id, title=None, body=None, state_event=Non
             # GitLab expects a comma-separated string for labels
             data["labels"] = ",".join(labels)
 
-        response = self.session.put(
-            url,
-            json=data,
-            timeout=self.default_timeout,
-        )
-        response.raise_for_status()
-        return response.json()
+        return self.put(url, json=data)
 
     def post_comment(self, repo_id, issue_id, comment_body):
         """Post a comment on an existing GitLab issue."""
         project_path = quote(repo_id, safe="")
         url = f"{self.api_url}/projects/{project_path}/issues/{issue_id}/notes"
         data = {"body": comment_body}
 
-        response = self.session.post(
-            url,
-            json=data,
-            timeout=self.default_timeout,
-        )
-        response.raise_for_status()
-        return response.json()
+        return self.post(url, json=data)
 
     @staticmethod
     def extract_gitlab_project_path(url):