[PM-10314] Auto-enable Single Org when a Domain is Verified
đī¸ Tracking
https://bitwarden.atlassian.net/browse/PM-10330
đ Objective
This is to allow for the auto enabling of the Single Org Policy when a claimed domain is verified. This will also block a user from disabling the Single Org Policy if the organization has any verified domains.
This also included a refatctor of the ValidateOrganizationDomain scheduled job to use the VerifyOrganizationDomainCommand. This also required a refactor of the command to allow for it to be called by user action or the background job.
â° Reminders before review
- Contributor guidelines followed
- All formatters and local linters executed and passed
- Written new unit and / or integration tests where applicable
- Protected functional changes with optionality (feature flags)
- Used internationalization (i18n) for all UI strings
- CI builds passed
- Communicated to DevOps any deployment requirements
- Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team
đĻŽ Reviewer guidelines
- đ (
:+1:) or similar for great changes - đ (
:memo:) or âšī¸ (:information_source:) for notes or general info - â (
:question:) for questions - đ¤ (
:thinking:) or đ (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion - đ¨ (
:art:) for suggestions / improvements - â (
:x:) or â ī¸ (:warning:) for more significant problems or concerns needing attention - đą (
:seedling:) or âģī¸ (:recycle:) for future improvements or indications of technical debt - â (
:pick:) for minor or nitpick changes
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 42.31%. Comparing base (
a952d10) to head (b4833b1). Report is 5 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #4897 +/- ##
==========================================
+ Coverage 42.29% 42.31% +0.02%
==========================================
Files 1384 1385 +1
Lines 64528 64550 +22
Branches 5930 5932 +2
==========================================
+ Hits 27289 27314 +25
+ Misses 36010 36007 -3
Partials 1229 1229
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One â Scan Summary & Details â 3c190cef-742b-4b4c-8cf0-f408dacb410d
New Issues
| Severity | Issue | Source File / Package | Checkmarx Insight |
|---|---|---|---|
![]() |
Privacy_Violation | /src/Core/Auth/Services/Implementations/AuthRequestService.cs: 213 | Attack Vector |
Fixed Issues
| Severity | Issue | Source File / Package |
|---|---|---|
![]() |
CSRF | /src/Billing/Controllers/RecoveryController.cs: 38 |
![]() |
CSRF | /src/Api/Vault/Controllers/CiphersController.cs: 107 |
![]() |
CSRF | /src/Api/Controllers/PushController.cs: 38 |
![]() |
CSRF | /src/Admin/AdminConsole/Controllers/OrganizationsController.cs: 351 |
![]() |
CSRF | /src/Billing/Controllers/StripeController.cs: 164 |
![]() |
CSRF | /src/Api/AdminConsole/Public/Controllers/MembersController.cs: 96 |
![]() |
CSRF | /src/Api/AdminConsole/Controllers/OrganizationUsersController.cs: 227 |
![]() |
CSRF | /src/Api/Vault/Controllers/CiphersController.cs: 107 |
![]() |
CSRF | /src/Api/AdminConsole/Controllers/PoliciesController.cs: 58 |
![]() |
CSRF | /src/Api/AdminConsole/Controllers/PoliciesController.cs: 75 |
![]() |
CSRF | /src/Api/AdminConsole/Public/Controllers/PoliciesController.cs: 42 |
![]() |
CSRF | /src/Api/AdminConsole/Public/Controllers/PoliciesController.cs: 62 |
![]() |
CSRF | /src/Admin/AdminConsole/Controllers/OrganizationsController.cs: 351 |
![]() |
CSRF | /src/Api/Controllers/PushController.cs: 38 |
![]() |
CSRF | /src/Api/Controllers/PushController.cs: 38 |
![]() |
Privacy_Violation | /src/Core/Auth/Services/Implementations/AuthRequestService.cs: 208 |
![]() |
Log_Forging | /src/Api/Vault/Controllers/CiphersController.cs: 170 |

