Skip to content

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

Open
trmartin4 wants to merge 1 commit intomainfrom
auth/remove-pm-22110-disable-alternate-login-methods
Open

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

Conversation

@trmartin4
Copy link
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

@github-actions
Copy link
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

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

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.

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