Skip to content

Use bitbucket static user for bitbucket git auth#3851

Open
SantiarErhi wants to merge 4 commits intoscala-steward-org:mainfrom
SantiarErhi:main
Open

Use bitbucket static user for bitbucket git auth#3851
SantiarErhi wants to merge 4 commits intoscala-steward-org:mainfrom
SantiarErhi:main

Conversation

@SantiarErhi
Copy link
Copy Markdown

Bitbucket App password will be disabled in 2.5 months (see https://www.atlassian.com/blog/bitbucket/bitbucket-cloud-enters-phase-2-of-app-password-deprecation). Since API token usage is inconsistent between bitbucket.org used for cloning repositories and api.bitbucket.org/2.0 used for working with PRs, etc (see https://support.atlassian.com/bitbucket-cloud/docs/using-api-tokens/), it's impossible to use API tokens with scala-steward and be able to clone a repository and read/write pull requests.

In order not to write another CLI option to specify another user for git or api, this solution uses static user name feature (see https://support.atlassian.com/bitbucket-cloud/docs/using-api-tokens/#Include-the-API-token-in-the-URL).

It works for me, but I'm not sure whether it's an appropriate solution.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.37%. Comparing base (e11ea69) to head (7599f18).
⚠️ Report is 17 commits behind head on main.

Files with missing lines Patch % Lines
...ala/org/scalasteward/core/forge/BasicAuthAlg.scala 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3851      +/-   ##
==========================================
- Coverage   90.39%   90.37%   -0.02%     
==========================================
  Files         177      177              
  Lines        5110     5111       +1     
  Branches      458      422      -36     
==========================================
  Hits         4619     4619              
- Misses        491      492       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SantiarErhi SantiarErhi marked this pull request as draft April 2, 2026 12:19
@SantiarErhi SantiarErhi marked this pull request as ready for review April 14, 2026 09:01
@SantiarErhi
Copy link
Copy Markdown
Author

Bitbucket auth logic has been moved to separate class in order to not change logic for ALL ForgeTypes using BasicAuthAlg.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant