[PM-11730] Remove feature flag: AC-2476-deprecate-stripe-sources-api

[amorask-bitwarden]

🎟️ Tracking

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

📔 Objective

Removes the Stripe Sources API Deprecation feature flag.

📸 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

[github-actions]

LaunchDarkly flag references

❌ 1 flag removed

Name	Key	Aliases found	Info
Remove Integration with Stripe's Sources API	AC-2476-deprecate-stripe-sources-api		✅ all references removed

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 44.22%. Comparing base (9e7d1ab) to head (189a449).

Files with missing lines	Patch %	Lines
src/Core/Services/Implementations/UserService.cs	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5201      +/-   ##
==========================================
+ Coverage   44.09%   44.22%   +0.12%     
==========================================
  Files        1476     1476              
  Lines       68273    68222      -51     
  Branches     6181     6171      -10     
==========================================
+ Hits        30106    30172      +66     
+ Misses      36857    36740     -117     
  Partials     1310     1310              

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

[github-actions]

Checkmarx One – Scan Summary & Details – 359c8485-9c26-4fc3-a29e-a0b4a68b6aba

New Issues (6)

Checkmarx found the following issues in this Pull Request

Severity	Issue	Source File / Package	Checkmarx Insight
	CSRF	/src/Billing/Controllers/PayPalController.cs: 66	details Method PostIpn at line 66 of /src/Billing/Controllers/PayPalController.cs gets a parameter from a user request from Body. This parameter value flow... Attack Vector
	Privacy_Violation	/src/Core/AdminConsole/OrganizationAuth/UpdateOrganizationAuthRequestCommand.cs: 84	details Method UpdateAsync at line 84 of /src/Core/AdminConsole/OrganizationAuth/UpdateOrganizationAuthRequestCommand.cs sends user information outside the... Attack Vector
	Privacy_Violation	/src/Core/NotificationHub/NotificationHubPushNotificationService.cs: 195	details Method PushAuthRequestAsync at line 195 of /src/Core/NotificationHub/NotificationHubPushNotificationService.cs sends user information outside the a... Attack Vector
	Privacy_Violation	/src/Api/Vault/Models/Request/CipherRequestModel.cs: 173	details Method ToCipherLoginData at line 173 of /src/Api/Vault/Models/Request/CipherRequestModel.cs sends user information outside the application. This ma... Attack Vector
	Privacy_Violation	/src/Api/Vault/Models/Request/CipherRequestModel.cs: 202	details Method ToCipherIdentityData at line 202 of /src/Api/Vault/Models/Request/CipherRequestModel.cs sends user information outside the application. This... Attack Vector
	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 (22)

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

Severity	Issue	Source File / Package
	CSRF	/src/Api/Tools/Controllers/OrganizationExportController.cs: 53
	CSRF	/src/Api/Auth/Controllers/AccountsController.cs: 470
	CSRF	/src/Api/AdminConsole/Public/Controllers/PoliciesController.cs: 65
	CSRF	/src/Api/AdminConsole/Public/Controllers/PoliciesController.cs: 46
	CSRF	/src/Admin/AdminConsole/Controllers/OrganizationsController.cs: 371
	CSRF	/src/Api/AdminConsole/Controllers/GroupsController.cs: 80
	CSRF	/src/Api/AdminConsole/Controllers/GroupsController.cs: 121
	Log_Forging	/src/Api/AdminConsole/Controllers/ProvidersController.cs: 72
	Log_Forging	/src/Api/Billing/Controllers/OrganizationBillingController.cs: 259
	Log_Forging	/src/Api/Billing/Controllers/OrganizationBillingController.cs: 259
	Log_Forging	/src/Api/Billing/Controllers/OrganizationBillingController.cs: 259
	Log_Forging	/src/Api/Auth/Controllers/AccountsController.cs: 639
	Log_Forging	/src/Api/Auth/Controllers/AccountsController.cs: 639
	Log_Forging	/src/Api/Auth/Controllers/AccountsController.cs: 639
	Log_Forging	/src/Api/Billing/Controllers/ProviderBillingController.cs: 104
	Log_Forging	/src/Api/Billing/Controllers/ProviderBillingController.cs: 104
	Log_Forging	/src/Api/Billing/Controllers/ProviderBillingController.cs: 104
	Log_Forging	/src/Api/AdminConsole/Controllers/OrganizationsController.cs: 185
	Log_Forging	/src/Api/AdminConsole/Controllers/OrganizationsController.cs: 170
	Log_Forging	/src/Api/AdminConsole/Controllers/ProvidersController.cs: 72
	Log_Forging	/src/Api/AdminConsole/Controllers/ProvidersController.cs: 72
	Log_Forging	/src/Api/AdminConsole/Controllers/ProvidersController.cs: 72

[amorask-bitwarden]

@cturnbull-bitwarden @JimmyVo16 Had some merge conflicts here if you two don't mind re-reviewing.