From 0cbf65dab6e66a35f4a61e36a31e266ae2080592 Mon Sep 17 00:00:00 2001 From: ChunTing Wu Date: Fri, 11 Apr 2025 14:59:27 +0800 Subject: [PATCH] Update pr_agent/git_providers/bitbucket_provider.py Co-authored-by: Prateek <110811408+Prateikx@users.noreply.github.com> --- pr_agent/git_providers/bitbucket_provider.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/pr_agent/git_providers/bitbucket_provider.py b/pr_agent/git_providers/bitbucket_provider.py index 3a0d20be..50d84abc 100644 --- a/pr_agent/git_providers/bitbucket_provider.py +++ b/pr_agent/git_providers/bitbucket_provider.py @@ -625,14 +625,15 @@ class BitbucketProvider(GitProvider): get_logger().error(f"repo_url_to_clone: {repo_url_to_clone} is not a valid bitbucket URL.") return None - if hasattr(self, 'basic_token'): + if self.auth_type == "basic": # Basic auth with token clone_url = f"{scheme}x-token-auth:{self.basic_token}@bitbucket.org{base_url}" - elif hasattr(self, 'bearer_token'): + elif self.auth_type == "bearer": # Bearer token clone_url = f"{scheme}x-token-auth:{self.bearer_token}@bitbucket.org{base_url}" else: - get_logger().error("No valid authentication method provided. Returning None") + # This case should ideally not be reached if __init__ validates auth_type + get_logger().error(f"Unsupported or uninitialized auth_type: {getattr(self, 'auth_type', 'N/A')}. Returning None") return None return clone_url