Skip to content

Conversation

@mickgmdb
Copy link
Collaborator

Fixed Bitbucket authenticated cloning bug

Copilot AI review requested due to automatic review settings November 25, 2025 07:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a bug in Bitbucket authenticated cloning by allowing token-only authentication without requiring a username. The change ensures that KF_BITBUCKET_TOKEN alone is sufficient for git operations using x-token-auth.

Key Changes:

  • Added fallback authentication path for token-only Bitbucket authentication
  • Updated credential checking logic to include bitbucket_token
  • Simplified documentation to reflect that username is no longer required for Workspace API tokens

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/git_binary.rs Added token-only authentication fallback and updated credential checking logic; includes comprehensive test coverage
README.md Removed requirement for KF_BITBUCKET_USERNAME when using Workspace API token
Cargo.toml Version bump to 1.68.0
CHANGELOG.md Added release notes for v1.68.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

README.md Outdated
- **Workspace API token (Cloud)** – set `KF_BITBUCKET_TOKEN`. `KF_BITBUCKET_USERNAME`
is optional; Kingfisher automatically uses the token for Bitbucket REST APIs
and authenticates git operations as `x-token-auth`.
- **Workspace API token (Cloud)** – set `KF_BITBUCKET_TOKEN`.Kingfisher automatically uses the token for Bitbucket REST APIs and authenticates git operations as `x-token-auth`.
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after period. Should be 'KF_BITBUCKET_TOKEN`. Kingfisher' with a space between the period and 'Kingfisher'.

Suggested change
- **Workspace API token (Cloud)** – set `KF_BITBUCKET_TOKEN`.Kingfisher automatically uses the token for Bitbucket REST APIs and authenticates git operations as `x-token-auth`.
- **Workspace API token (Cloud)** – set `KF_BITBUCKET_TOKEN`. Kingfisher automatically uses the token for Bitbucket REST APIs and authenticates git operations as `x-token-auth`.
Copilot uses AI. Check for mistakes.
@mickgmdb mickgmdb merged commit 3f9b3df into main Nov 25, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants