Skip to content

[PM-35192] feat: Add associated-domains entitlements for FedRAMP environment#2852

Merged
fedemkr merged 1 commit into
mainfrom
worktree-PM-35192
Jul 2, 2026
Merged

[PM-35192] feat: Add associated-domains entitlements for FedRAMP environment#2852
fedemkr merged 1 commit into
mainfrom
worktree-PM-35192

Conversation

@fedemkr

@fedemkr fedemkr commented Jul 2, 2026

Copy link
Copy Markdown
Member

🎟️ Tracking

PM-35192

📔 Objective

Adds webcredentials:*.bitwarden-gov.com and applinks:*.bitwarden-gov.com entries to the com.apple.developer.associated-domains array in Bitwarden.entitlements, so AutoFill password/passkey suggestions and universal links work against the new FedRAMP government cloud environment (vault.bitwarden-gov.com), matching the existing pattern already in place for bitwarden.com, bitwarden.eu, and bitwarden.pw.

Part of epic PM-35110 ([Mobile] Support FedRAMP environment selection).

📸 Screenshots

N/A — no UI change.

…ronment

Adds webcredentials and applinks entries for *.bitwarden-gov.com so
AutoFill password suggestions and universal links work against the new
FedRAMP government cloud environment, matching the existing pattern for
bitwarden.com/eu/pw. Part of epic PM-35110 (Mobile FedRAMP support).
@fedemkr fedemkr added the ai-review Request a Claude code review label Jul 2, 2026
@github-actions github-actions Bot added app:password-manager Bitwarden Password Manager app context t:feature labels Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Reviewed the addition of webcredentials:*.bitwarden-gov.com and applinks:*.bitwarden-gov.com to the main app's com.apple.developer.associated-domains entitlement. The new entries follow the exact existing pattern for bitwarden.com, bitwarden.eu, and bitwarden.pw, enabling AutoFill and universal links against the FedRAMP vault.bitwarden-gov.com environment. Confirmed the main app is the only entitlements file declaring associated-domains, so the AutoFill extension correctly inherits credential-provider matching and needs no parallel change. The plist structure remains valid and no other files require updating for this infrastructure-prep step.

No findings.

@fedemkr
fedemkr marked this pull request as ready for review July 2, 2026 21:32
@fedemkr
fedemkr requested review from a team and matt-livefront as code owners July 2, 2026 21:32
@fedemkr
fedemkr enabled auto-merge (squash) July 2, 2026 21:32
@fedemkr fedemkr added the enhancement New feature or request label Jul 2, 2026
@fedemkr
fedemkr merged commit 4680d92 into main Jul 2, 2026
20 of 27 checks passed
@fedemkr
fedemkr deleted the worktree-PM-35192 branch July 2, 2026 22:08
@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.16%. Comparing base (9b721b3) to head (8f9bf93).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2852      +/-   ##
==========================================
+ Coverage   81.14%   81.16%   +0.02%     
==========================================
  Files        1026     1026              
  Lines       65978    65983       +5     
==========================================
+ Hits        53539    53558      +19     
+ Misses      12439    12425      -14     

☔ View full report in Codecov by Harness.
📢 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.

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

Labels

ai-review Request a Claude code review app:password-manager Bitwarden Password Manager app context enhancement New feature or request t:feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants