Skip to content

SDK Update - com.bitwarden:sdk-android 2.0.0-6340-00b609f9#6823

Merged
david-livefront merged 1 commit intomainfrom
bitwarden-sdk-update
Apr 22, 2026
Merged

SDK Update - com.bitwarden:sdk-android 2.0.0-6340-00b609f9#6823
david-livefront merged 1 commit intomainfrom
bitwarden-sdk-update

Conversation

@david-livefront
Copy link
Copy Markdown
Collaborator

🎟️ Tracking

N/A

📔 Objective

Updates the SDK version from 2.0.0-6312-3e1ead66 to com.bitwarden:sdk-android 2.0.0-6340-00b609f9

@david-livefront david-livefront requested a review from a team as a code owner April 22, 2026 19:50
@github-actions github-actions Bot added app:password-manager Bitwarden Password Manager app context app:authenticator Bitwarden Authenticator app context t:deps Change Type - Dependencies labels Apr 22, 2026
) : ServerCommunicationConfigRepository {

override suspend fun get(hostname: String): ServerCommunicationConfig? {
override suspend fun get(domain: String): ServerCommunicationConfig? {
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I am unsure if this is a functional change or if they are actually passing in a different value.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

bitwarden/sdk-internal#820

The hostname parameter has been renamed to domain in all public APIs (repository trait, platform API trait, client methods, WASM bindings, UniFFI bindings). This is a breaking change for existing implementations of the repository and platform API traits.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2026

Codecov Report

❌ Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.58%. Comparing base (985562e) to head (ce946e8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...ository/ServerCommunicationConfigRepositoryImpl.kt 70.00% 0 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6823      +/-   ##
==========================================
- Coverage   85.58%   85.58%   -0.01%     
==========================================
  Files         830      830              
  Lines       61506    61509       +3     
  Branches     8590     8592       +2     
==========================================
+ Hits        52641    52642       +1     
  Misses       5900     5900              
- Partials     2965     2967       +2     
Flag Coverage Δ
app-data 17.35% <70.00%> (+<0.01%) ⬆️
app-ui-auth-tools 20.32% <0.00%> (-0.01%) ⬇️
app-ui-platform 15.51% <0.00%> (-0.01%) ⬇️
app-ui-vault 25.89% <0.00%> (-0.01%) ⬇️
authenticator 6.69% <0.00%> (+0.02%) ⬆️
lib-core-network-bridge 4.20% <0.00%> (-0.01%) ⬇️
lib-data-ui 1.00% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@david-livefront
Copy link
Copy Markdown
Collaborator Author

Thanks @SaintPatrck

@david-livefront david-livefront added this pull request to the merge queue Apr 22, 2026
Merged via the queue into main with commit c01fc62 Apr 22, 2026
26 of 28 checks passed
@david-livefront david-livefront deleted the bitwarden-sdk-update branch April 22, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app:authenticator Bitwarden Authenticator app context app:password-manager Bitwarden Password Manager app context t:deps Change Type - Dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants