Skip to content

[PM-30584] Add unlock for key connector with SDK#19367

Open
quexten wants to merge 20 commits intomainfrom
km/unlock-kc-with-sdk
Open

[PM-30584] Add unlock for key connector with SDK#19367
quexten wants to merge 20 commits intomainfrom
km/unlock-kc-with-sdk

Conversation

@quexten
Copy link
Copy Markdown
Contributor

@quexten quexten commented Mar 4, 2026

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-30584

📔 Objective

Adds unlock via the unlock service in the login strategies. The unlock service implements key-connector-unlock via the SDK.

📸 Screenshots

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 4, 2026

Logo
Checkmarx One – Scan Summary & Detailsb6a6bb9b-4f03-473f-adca-f7779b3eeb8b

Great job! No new security vulnerabilities introduced in this pull request

@quexten quexten changed the title Add unlock for key connector with SDK [PM-30584] Add unlock for key connector with SDK Mar 11, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 61.76471% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.52%. Comparing base (e97450a) to head (6669356).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
libs/unlock/src/default-unlock.service.ts 0.00% 8 Missing ⚠️
...rc/auth/models/response/identity-token.response.ts 0.00% 4 Missing ⚠️
libs/unlock/src/index.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #19367      +/-   ##
==========================================
+ Coverage   45.44%   45.52%   +0.08%     
==========================================
  Files        3768     3773       +5     
  Lines      110969   111416     +447     
  Branches    16910    16919       +9     
==========================================
+ Hits        50428    50721     +293     
- Misses      58321    58475     +154     
  Partials     2220     2220              

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@quexten quexten marked this pull request as ready for review March 11, 2026 13:04
@quexten quexten requested review from a team as code owners March 11, 2026 13:04
@ike-kottlowski
Copy link
Copy Markdown
Contributor

#19422 has been merged.

mzieniukbw
mzieniukbw previously approved these changes Mar 16, 2026
mzieniukbw
mzieniukbw previously approved these changes Mar 17, 2026
@sonarqubecloud
Copy link
Copy Markdown

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.

3 participants