PM-16261 move ImportCiphersAsync to the tools team

[prograhamming]

🎟️ Tracking

https://bitwarden.atlassian.net/jira/software/c/projects/BW/boards/92?selectedIssue=PM-16261

📔 Objective

Currently the ImportCiphersAsync methods located in the CipherService are used by the ImportCiphersController. The import process is owned by Tools but the file-structure does not reflect this, which requires reviews from Vault-team anytime we need to adjust something.

📸 Screenshots

⏰ 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]

Codecov Report

Attention: Patch coverage is 81.88976% with 23 lines in your changes missing coverage. Please review.

Project coverage is 44.10%. Comparing base (04402c1) to head (1689d90).
Report is 33 commits behind head on main.

Files with missing lines	Patch %	Lines
.../Core/Tools/ImportFeatures/ImportCiphersCommand.cs	83.89%	12 Missing and 7 partials ⚠️
...c/Api/Tools/Controllers/ImportCiphersController.cs	0.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5245      +/-   ##
==========================================
+ Coverage   44.02%   44.10%   +0.07%     
==========================================
  Files        1476     1478       +2     
  Lines       68291    68316      +25     
  Branches     6175     6175              
==========================================
+ Hits        30065    30128      +63     
+ Misses      36919    36877      -42     
- Partials     1307     1311       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Checkmarx One – Scan Summary & Details – bbcae1aa-bee3-48b7-8ee6-65fabc72d445

New Issues (1)

Checkmarx found the following issues in this Pull Request

Severity	Issue	Source File / Package	Checkmarx Insight
	Missing_CSP_Header	/src/Core/MailTemplates/Handlebars/AdminConsole/DomainClaimedByOrganization.html.hbs: 7	details A Content Security Policy is not explicitly defined within the web-application. Attack Vector

Fixed Issues (4)

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

Severity	Issue	Source File / Package
	CSRF	/src/Api/AdminConsole/Public/Controllers/PoliciesController.cs: 46
	CSRF	/src/Api/AdminConsole/Public/Controllers/PoliciesController.cs: 65
	CSRF	/src/Identity/Controllers/AccountsController.cs: 119
	CSRF	/src/Api/Tools/Controllers/SendsController.cs: 147

[djsmith85]

@prograhamming: Great work! Changes are looking good overall. Just some small things I noticed