[SM-716] Adding ability for service account to have write access

[cd-bitwarden]

Type of change

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

Objective

Service accounts need to be able to have write access to projects and secrets

Code changes

bitwarden_license/src/Commercial.Core/SecretsManager/AuthorizationHandlers/Projects/ProjectAuthorizationHandler.cs - allow service accounts

bitwarden_license/src/Commercial.Core/SecretsManager/AuthorizationHandlers/Secrets/SecretAuthorizationHandler.cs - remove code that disallows service accounts

bitwarden_license/src/Commercial.Core/SecretsManager/Commands/Projects/CreateProjectCommand.cs - bootstrapping service accounts to have access to projects they create

src/Api/SecretsManager/Controllers/ProjectsController.cs - pass to the function what type of user is logged in.

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

[Thomas-Avery]

Overall, updates look good.

Looks like unit tests are throwing errors on build, and we will need to update the authorization unit tests with these changes.

There are a few small things to address within the code below.

[Thomas-Avery]

This looks good.

Let's add to the unit tests for ProjectAuthorizationHandlerTestswith a new permission type PermissionType.RunAsServiceAccountWithPermission so we have test coverage for these changes.

[cd-bitwarden]

This looks good.

Let's add to the unit tests for ProjectAuthorizationHandlerTestswith a new permission type PermissionType.RunAsServiceAccountWithPermission so we have test coverage for these changes.

Great Idea Thomas! I added them :D

[Thomas-Avery]

LGTM

[Thomas-Avery]

Looks good, thank you!