Skip to content

[PM-22110] Remove pm-22110-disable-alternate-login-methods feature flag#7274

Merged
trmartin4 merged 1 commit intomainfrom
auth/remove-pm-22110-disable-alternate-login-methods
Mar 25, 2026
Merged

[PM-22110] Remove pm-22110-disable-alternate-login-methods feature flag#7274
trmartin4 merged 1 commit intomainfrom
auth/remove-pm-22110-disable-alternate-login-methods

Conversation

@trmartin4
Copy link
Copy Markdown
Member

@trmartin4 trmartin4 commented Mar 22, 2026

🎟️ Tracking

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

📔 Objective

Removes pm-22110-disable-alternate-login-methods feature flag from server.

This flag was not ever used on the server and was removed from clients as follows:

Location Status Details
Clients ✅ Removed 23ac477 (2025-11-21)

Timeline

Milestone Version
First client without flag 2026.1.0
Last client with flag 2025.11.1
Server safe to remove 2026.2.0 (last client + 3 months)
Current server version v2026.3.0

📸 Screenshots

@trmartin4 trmartin4 changed the title Remove flag from Constants [PM-22110] Remove pm-22110-disable-alternate-login-methods feature flag Mar 22, 2026
@trmartin4 trmartin4 marked this pull request as ready for review March 22, 2026 20:09
@trmartin4 trmartin4 requested review from a team and ike-kottlowski March 22, 2026 20:10
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

Logo
Checkmarx One – Scan Summary & Detailsd13f2060-90d5-419b-bdea-3e8db54647e9


New Issues (1) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 HIGH CVE-2026-32933 Nuget-AutoMapper-12.0.1
detailsRecommended version: 15.1.1
Description: AutoMapper is vulnerable to a Denial-of-Service (DoS) attack. Versions prior to 15.1.1 and 16.x prior to 16.1.1, when mapping deeply nested object ...
Attack Vector: NETWORK
Attack Complexity: LOW
Vulnerable Package

Fixed Issues (1) Great job! The following issues were fixed in this Pull Request
Severity Issue Source File / Package
HIGH Cxb9102459-6a3a Nuget-AutoMapper-12.0.1

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.72%. Comparing base (514d8a1) to head (5d16281).
⚠️ Report is 149 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7274   +/-   ##
=======================================
  Coverage   57.72%   57.72%           
=======================================
  Files        2042     2042           
  Lines       89817    89817           
  Branches     7985     7985           
=======================================
  Hits        51844    51844           
  Misses      36115    36115           
  Partials     1858     1858           

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@trmartin4 trmartin4 merged commit 0130c60 into main Mar 25, 2026
46 checks passed
@trmartin4 trmartin4 deleted the auth/remove-pm-22110-disable-alternate-login-methods branch March 25, 2026 13:34
@djsmith85 djsmith85 added the t:feature Change Type - Feature Development label Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants