Skip to content

[PM-33037] Create feature flag pm-31885-send-controls for Send Policy MS 1#7134

Merged
harr1424 merged 2 commits intomainfrom
PM-33037-Create-feature-flag-pm-31885-send-controls-for-Send-Policy-MS-1
Mar 13, 2026
Merged

[PM-33037] Create feature flag pm-31885-send-controls for Send Policy MS 1#7134
harr1424 merged 2 commits intomainfrom
PM-33037-Create-feature-flag-pm-31885-send-controls-for-Send-Policy-MS-1

Conversation

@harr1424
Copy link
Copy Markdown
Contributor

@harr1424 harr1424 commented Mar 4, 2026

🎟️ Tracking

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

📔 Objective

This PR introduces a feature flag that will be used to gate changes to Send Policies.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 4, 2026

Logo
Checkmarx One – Scan Summary & Detailsa736caf1-d11a-4723-9f5a-5da647674829


New Issues (2) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 HIGH Path_Traversal /src/Api/Controllers/SelfHosted/SelfHostedOrganizationLicensesController.cs: 56
detailsMethod at line 56 of /src/Api/Controllers/SelfHosted/SelfHostedOrganizationLicensesController.cs gets dynamic data from the model element. This ...
Attack Vector
2 MEDIUM CSRF /src/Api/Auth/Controllers/AccountsController.cs: 291
detailsMethod at line 291 of /src/Api/Auth/Controllers/AccountsController.cs gets a parameter from a user request from model. This parameter value flow...
Attack Vector

@harr1424 harr1424 requested a review from a team March 4, 2026 01:08
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.43%. Comparing base (79089a0) to head (d88a95e).
⚠️ Report is 139 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7134   +/-   ##
=======================================
  Coverage   57.43%   57.43%           
=======================================
  Files        2032     2032           
  Lines       89366    89366           
  Branches     7947     7947           
=======================================
+ Hits        51328    51329    +1     
+ Misses      36193    36192    -1     
  Partials     1845     1845           

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

@sonarqubecloud
Copy link
Copy Markdown

@harr1424 harr1424 removed the needs-qa label Mar 13, 2026
@harr1424 harr1424 merged commit 864e356 into main Mar 13, 2026
42 of 43 checks passed
@harr1424 harr1424 deleted the PM-33037-Create-feature-flag-pm-31885-send-controls-for-Send-Policy-MS-1 branch March 13, 2026 21:48
@djsmith85 djsmith85 added the t:feature Change Type - Feature Development label Apr 3, 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.

4 participants