[PM-28346] Use SDK for attachment delete operations

[gbubemismith]

🎟️ Tracking

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

📔 Objective

Added feature flag constant for attachment sdk operations

📸 Screenshots

[github-actions]

Checkmarx One – Scan Summary & Details – 27886a5c-8d65-42d3-b8ef-622b64298f0b

---

New Issues (4)

Checkmarx found the following issues in this Pull Request

#	Severity	Issue	Source File / Package	Checkmarx Insight
1		CSRF	src/Api/Vault/Controllers/CiphersController.cs: 1558	details Method at line 1558 of /src/Api/Vault/Controllers/CiphersController.cs gets a parameter from a user request from id. This parameter value flows ... Attack Vector
2		CSRF	src/Api/Auth/Controllers/AccountsController.cs: 291	details Method 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
3		CSRF	src/Api/Auth/Controllers/AccountsController.cs: 452	details Method at line 452 of /src/Api/Auth/Controllers/AccountsController.cs gets a parameter from a user request from model. This parameter value flow... Attack Vector
4		CSRF	src/Api/Vault/Controllers/CiphersController.cs: 1385	details Method at line 1385 of /src/Api/Vault/Controllers/CiphersController.cs gets a parameter from a user request from id. This parameter value flows ... Attack Vector

---

Fixed Issues (4)

Great job! The following issues were fixed in this Pull Request

Severity	Issue	Source File / Package
	CSRF	src/Api/KeyManagement/Controllers/AccountsKeyManagementController.cs: 145
	CSRF	src/Api/KeyManagement/Controllers/AccountsKeyManagementController.cs: 97
	CSRF	src/Identity/Controllers/AccountsController.cs: 138
	CSRF	src/Api/KeyManagement/Controllers/AccountsKeyManagementController.cs: 173

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.36%. Comparing base (c281161) to head (8b30e39).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7538      +/-   ##
==========================================
- Coverage   63.81%   59.36%   -4.46%     
==========================================
  Files        2087     2087              
  Lines       92262    92262              
  Branches     8200     8200              
==========================================
- Hits        58880    54770    -4110     
- Misses      31356    35553    +4197     
+ Partials     2026     1939      -87     

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