[PM-3084] fix: Fix null handling in SendLicenseExpiredAsync method

[GSWXXN]

Type of change

- [x] Bug fix
- [ ] New feature development
- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc)
- [ ] Build/deploy pipeline (DevOps)
- [ ] Other

Objective

Fixed that when the user license is not available, there may be an error when logging in to Bitwarden, and the user may not receive the license expiration reminder email

Code changes

• HandlebarsMailService.cs: A null check has been added to prevent an exception when passing an empty organizationName to the CoreHelpers.SanitizeForEmail() method. Without this change, the specific location in the code (at this link) would throw an exception.

Before you submit

• Please check for formatting errors (dotnet format --verify-no-changes) (required)
• If making database changes - make sure you also update Entity Framework queries and/or migrations
• Please add unit tests where it makes sense to do so (encouraged but not required)
• If this change requires a documentation update - notify the documentation team
• If this change has particular deployment requirements - notify the DevOps team

[bitwarden-bot]

Thank you for your contribution! We've added this to our internal Community PR board for review.
ID: PM-3084

[bitwarden-bot]

Checkmarx One – Scan Summary & Details – 16678278-5020-41d3-a366-6937533db8d4

Fixed Issues

Severity	Issue	Source File / Package
	Passwords And Secrets - Generic Password	/test-database.yml: 173
	Passwords And Secrets - Generic Password	/test-database.yml: 111
	Passwords And Secrets - Generic Password	/test-database.yml: 80
	Passwords And Secrets - Generic Password	/test-database.yml: 69
	Passwords And Secrets - Generic Password	/test-database.yml: 105
	Passwords And Secrets - Generic Password	/test-database.yml: 92
	Passwords And Secrets - Generic Password	/test-database.yml: 170
	Passwords And Secrets - Generic Password	/test-database.yml: 177
	CSRF	/src/Api/AdminConsole/Controllers/OrganizationsController.cs: 504
	CSRF	/src/Api/AdminConsole/Controllers/OrganizationsController.cs: 471
	CSRF	/src/Api/AdminConsole/Controllers/OrganizationsController.cs: 529
	CSRF	/src/Api/AdminConsole/Controllers/OrganizationUsersController.cs: 561
	CSRF	/src/Api/AdminConsole/Controllers/OrganizationsController.cs: 177
	CSRF	/src/Api/Billing/Public/Controllers/OrganizationController.cs: 47
	CSRF	/src/Api/AdminConsole/Controllers/OrganizationUsersController.cs: 279
	CSRF	/src/Api/AdminConsole/Controllers/OrganizationsController.cs: 504
	CSRF	/src/Api/AdminConsole/Controllers/OrganizationUsersController.cs: 547
	Unpinned Actions Full Length Commit SHA	/build.yml: 598
	Unpinned Actions Full Length Commit SHA	/build.yml: 548
	Unpinned Actions Full Length Commit SHA	/build.yml: 515
	Log_Forging	/src/Api/Auth/Controllers/EmergencyAccessController.cs: 159

[nsbgit]

LGTM

[amorask-bitwarden]

@GSWXXN Appreciate the contribution! I've reviewed this and since it's just a null-check addition, I'm going to push it through. You can expect this change to be in the release on 9/17.

Thanks again!