initial commit (#2400)

Author: yashv6655

integrations/github/src/haystack_integrations/components/connectors/github/file_editor.py

@@ -115,7 +115,9 @@ def _get_request_headers(self) -> dict:
         """
         headers = self.base_headers.copy()
         if self.github_token is not None:
-            headers["Authorization"] = f"Bearer {self.github_token.resolve_value()}"
+            token_value = self.github_token.resolve_value()
+            if token_value:
+                headers["Authorization"] = f"Bearer {token_value}"
         return headers
 
     def _get_file_content(self, owner: str, repo: str, path: str, branch: str) -> tuple[str, str]:

integrations/github/src/haystack_integrations/components/connectors/github/issue_commenter.py

@@ -67,7 +67,9 @@ def _get_request_headers(self) -> dict:
         """
         headers = self.base_headers.copy()
         if self.github_token is not None:
-            headers["Authorization"] = f"Bearer {self.github_token.resolve_value()}"
+            token_value = self.github_token.resolve_value()
+            if token_value:
+                headers["Authorization"] = f"Bearer {token_value}"
         return headers
 
     def _parse_github_url(self, url: str) -> tuple[str, str, int]:

integrations/github/src/haystack_integrations/components/connectors/github/issue_viewer.py

@@ -66,7 +66,9 @@ def _get_request_headers(self) -> dict:
         """
         headers = self.base_headers.copy()
         if self.github_token:
-            headers["Authorization"] = f"Bearer {self.github_token.resolve_value()}"
+            token_value = self.github_token.resolve_value()
+            if token_value:
+                headers["Authorization"] = f"Bearer {token_value}"
         return headers
 
     def _parse_github_url(self, url: str) -> tuple[str, str, int]:

integrations/github/src/haystack_integrations/components/connectors/github/pr_creator.py

@@ -65,7 +65,9 @@ def _get_request_headers(self) -> dict:
         """
         headers = self.base_headers.copy()
         if self.github_token is not None:
-            headers["Authorization"] = f"Bearer {self.github_token.resolve_value()}"
+            token_value = self.github_token.resolve_value()
+            if token_value:
+                headers["Authorization"] = f"Bearer {token_value}"
         return headers
 
     def _parse_issue_url(self, issue_url: str) -> tuple[str, str, str]:

integrations/github/src/haystack_integrations/components/connectors/github/repo_forker.py

@@ -85,7 +85,9 @@ def _get_request_headers(self) -> dict:
         """
         headers = self.base_headers.copy()
         if self.github_token is not None:
-            headers["Authorization"] = f"Bearer {self.github_token.resolve_value()}"
+            token_value = self.github_token.resolve_value()
+            if token_value:
+                headers["Authorization"] = f"Bearer {token_value}"
         return headers
 
     def _parse_github_url(self, url: str) -> tuple[str, str, str]:

integrations/github/src/haystack_integrations/components/connectors/github/repo_viewer.py

@@ -109,7 +109,9 @@ def _get_request_headers(self) -> dict:
         """
         headers = self.base_headers.copy()
         if self.github_token is not None:
-            headers["Authorization"] = f"Bearer {self.github_token.resolve_value()}"
+            token_value = self.github_token.resolve_value()
+            if token_value:
+                headers["Authorization"] = f"Bearer {token_value}"
         return headers
 
     def to_dict(self) -> Dict[str, Any]:

integrations/github/tests/test_file_editor.py

@@ -269,3 +269,14 @@ def test_run_error_handling(self, mock_get, monkeypatch):
                 repo="owner/repo",
                 branch="main",
             )
+
+    def test_get_request_headers_with_empty_token(self, monkeypatch):
+        monkeypatch.setenv("GITHUB_TOKEN", "")
+
+        editor = GitHubFileEditor()
+
+        headers = editor._get_request_headers()
+
+        assert "Authorization" not in headers
+        assert headers["Accept"] == "application/vnd.github.v3+json"
+        assert headers["User-Agent"] == "Haystack/GitHubFileEditor"

integrations/github/tests/test_issue_commenter.py

@@ -114,3 +114,14 @@ def test_parse_github_url(self, monkeypatch):
 
         with pytest.raises(ValueError):
             commenter._parse_github_url("https://github.com/invalid/url")
+
+    def test_get_request_headers_with_empty_token(self, monkeypatch):
+        monkeypatch.setenv("GITHUB_TOKEN", "")
+
+        commenter = GitHubIssueCommenter()
+
+        headers = commenter._get_request_headers()
+
+        assert "Authorization" not in headers
+        assert headers["Accept"] == "application/vnd.github.v3+json"
+        assert headers["User-Agent"] == "Haystack/GitHubIssueCommenter"

integrations/github/tests/test_issue_viewer.py

@@ -150,3 +150,37 @@ def test_parse_github_url(self, monkeypatch):
 
         with pytest.raises(ValueError):
             viewer._parse_github_url("https://github.com/invalid/url")
+
+    def test_get_request_headers_with_valid_token(self, monkeypatch):
+        monkeypatch.setenv("GITHUB_TOKEN", "test-token-value")
+
+        token = Secret.from_env_var("GITHUB_TOKEN")
+        viewer = GitHubIssueViewer(github_token=token)
+
+        headers = viewer._get_request_headers()
+
+        assert "Authorization" in headers
+        assert headers["Authorization"] == "Bearer test-token-value"
+        assert headers["Accept"] == "application/vnd.github.v3+json"
+        assert headers["User-Agent"] == "Haystack/GitHubIssueViewer"
+
+    def test_get_request_headers_without_token(self):
+        viewer = GitHubIssueViewer(github_token=None)
+
+        headers = viewer._get_request_headers()
+
+        assert "Authorization" not in headers
+        assert headers["Accept"] == "application/vnd.github.v3+json"
+        assert headers["User-Agent"] == "Haystack/GitHubIssueViewer"
+
+    def test_get_request_headers_with_empty_token(self, monkeypatch):
+        monkeypatch.setenv("GITHUB_TOKEN", "")
+
+        token = Secret.from_env_var("GITHUB_TOKEN")
+        viewer = GitHubIssueViewer(github_token=token)
+
+        headers = viewer._get_request_headers()
+
+        assert "Authorization" not in headers
+        assert headers["Accept"] == "application/vnd.github.v3+json"
+        assert headers["User-Agent"] == "Haystack/GitHubIssueViewer"

integrations/github/tests/test_pr_creator.py

@@ -132,3 +132,14 @@ def test_run_error_handling(self, _, mock_get, monkeypatch):
                 branch="feature-branch",
                 base="main",
             )
+
+    def test_get_request_headers_with_empty_token(self, monkeypatch):
+        monkeypatch.setenv("GITHUB_TOKEN", "")
+
+        pr_creator = GitHubPRCreator()
+
+        headers = pr_creator._get_request_headers()
+
+        assert "Authorization" not in headers
+        assert headers["Accept"] == "application/vnd.github.v3+json"
+        assert headers["User-Agent"] == "Haystack/GitHubPRCreator"