Merge branch 'main' into notifications-service-tests

Directory.Build.props

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>net8.0</TargetFramework>
 
-    <Version>2025.2.1</Version>
+    <Version>2025.2.3</Version>
 
     <RootNamespace>Bit.$(MSBuildProjectName)</RootNamespace>
     <ImplicitUsings>enable</ImplicitUsings>

bitwarden_license/src/Commercial.Core/Billing/ProviderBillingService.cs

@@ -475,20 +475,17 @@ public class ProviderBillingService(
         Provider provider,
         TaxInfo taxInfo)
     {
-        ArgumentNullException.ThrowIfNull(provider);
-        ArgumentNullException.ThrowIfNull(taxInfo);
-
-        if (string.IsNullOrEmpty(taxInfo.BillingAddressCountry) ||
-            string.IsNullOrEmpty(taxInfo.BillingAddressPostalCode))
+        if (taxInfo is not
+            {
+                BillingAddressCountry: not null and not "",
+                BillingAddressPostalCode: not null and not ""
+            })
         {
             logger.LogError("Cannot create customer for provider ({ProviderID}) without both a country and postal code", provider.Id);
-
             throw new BillingException();
         }
 
-        var providerDisplayName = provider.DisplayName();
-
-        var customerCreateOptions = new CustomerCreateOptions
+        var options = new CustomerCreateOptions
         {
             Address = new AddressOptions
             {
@@ -508,9 +505,9 @@ public class ProviderBillingService(
                     new CustomerInvoiceSettingsCustomFieldOptions
                     {
                         Name = provider.SubscriberType(),
-                        Value = providerDisplayName?.Length <= 30
-                            ? providerDisplayName
-                            : providerDisplayName?[..30]
+                        Value = provider.DisplayName()?.Length <= 30
+                            ? provider.DisplayName()
+                            : provider.DisplayName()?[..30]
                     }
                 ]
             },
@@ -522,7 +519,8 @@ public class ProviderBillingService(
 
         if (!string.IsNullOrEmpty(taxInfo.TaxIdNumber))
         {
-            var taxIdType = taxService.GetStripeTaxCode(taxInfo.BillingAddressCountry,
+            var taxIdType = taxService.GetStripeTaxCode(
+                taxInfo.BillingAddressCountry,
                 taxInfo.TaxIdNumber);
 
             if (taxIdType == null)
@@ -533,15 +531,20 @@ public class ProviderBillingService(
                 throw new BadRequestException("billingTaxIdTypeInferenceError");
             }
 
-            customerCreateOptions.TaxIdData =
+            options.TaxIdData =
             [
                 new CustomerTaxIdDataOptions { Type = taxIdType, Value = taxInfo.TaxIdNumber }
             ];
         }
 
+        if (!string.IsNullOrEmpty(provider.DiscountId))
+        {
+            options.Coupon = provider.DiscountId;
+        }
+
         try
         {
-            return await stripeAdapter.CustomerCreateAsync(customerCreateOptions);
+            return await stripeAdapter.CustomerCreateAsync(options);
         }
         catch (StripeException stripeException) when (stripeException.StripeError?.Code == StripeConstants.ErrorCodes.TaxIdInvalid)
         {

bitwarden_license/test/Commercial.Core.Test/Billing/ProviderBillingServiceTests.cs

@@ -731,18 +731,6 @@ public class ProviderBillingServiceTests
 
     #region SetupCustomer
 
-    [Theory, BitAutoData]
-    public async Task SetupCustomer_NullProvider_ThrowsArgumentNullException(
-        SutProvider<ProviderBillingService> sutProvider,
-        TaxInfo taxInfo) =>
-        await Assert.ThrowsAsync<ArgumentNullException>(() => sutProvider.Sut.SetupCustomer(null, taxInfo));
-
-    [Theory, BitAutoData]
-    public async Task SetupCustomer_NullTaxInfo_ThrowsArgumentNullException(
-        SutProvider<ProviderBillingService> sutProvider,
-        Provider provider) =>
-        await Assert.ThrowsAsync<ArgumentNullException>(() => sutProvider.Sut.SetupCustomer(provider, null));
-
     [Theory, BitAutoData]
     public async Task SetupCustomer_MissingCountry_ContactSupport(
         SutProvider<ProviderBillingService> sutProvider,

src/Admin/AdminConsole/Models/CreateMspProviderModel.cs

@@ -10,6 +10,9 @@ public class CreateMspProviderModel : IValidatableObject
     [Display(Name = "Owner Email")]
     public string OwnerEmail { get; set; }
 
+    [Display(Name = "Subscription Discount")]
+    public string DiscountId { get; set; }
+
     [Display(Name = "Teams (Monthly) Seat Minimum")]
     public int TeamsMonthlySeatMinimum { get; set; }
 
@@ -20,7 +23,8 @@ public class CreateMspProviderModel : IValidatableObject
     {
         return new Provider
         {
-            Type = ProviderType.Msp
+            Type = ProviderType.Msp,
+            DiscountId = DiscountId
         };
     }
 

src/Admin/AdminConsole/Views/Providers/CreateMsp.cshtml

@@ -1,3 +1,4 @@
+@using Bit.Core.Billing.Constants
 @model CreateMspProviderModel
 
 @{
@@ -12,6 +13,19 @@
             <label asp-for="OwnerEmail" class="form-label"></label>
             <input type="text" class="form-control" asp-for="OwnerEmail">
         </div>
+        <div class="mb-3">
+            @{
+                var selectList = new List<SelectListItem>
+                {
+                    new ("No discount", string.Empty, true),
+                    new ("20% - Open", StripeConstants.CouponIDs.MSPDiscounts.Open),
+                    new ("35% - Silver", StripeConstants.CouponIDs.MSPDiscounts.Silver),
+                    new ("50% - Gold", StripeConstants.CouponIDs.MSPDiscounts.Gold)
+                };
+            }
+            <label asp-for="DiscountId" class="form-label"></label>
+            <select class="form-select" asp-for="DiscountId" asp-items="selectList"></select>
+        </div>
         <div class="row">
             <div class="col-sm">
                 <div class="mb-3">

src/Admin/Controllers/UsersController.cs

@@ -102,12 +102,13 @@ public class UsersController : Controller
             return RedirectToAction("Index");
         }
 
-        var ciphers = await _cipherRepository.GetManyByUserIdAsync(id);
+        var ciphers = await _cipherRepository.GetManyByUserIdAsync(id, withOrganizations: false);
         var billingInfo = await _paymentService.GetBillingAsync(user);
         var billingHistoryInfo = await _paymentService.GetBillingHistoryAsync(user);
         var isTwoFactorEnabled = await _twoFactorIsEnabledQuery.TwoFactorIsEnabledAsync(user);
         var verifiedDomain = await AccountDeprovisioningEnabled(user.Id);
         var deviceVerificationRequired = await _userService.ActiveNewDeviceVerificationException(user.Id);
+
         return View(new UserEditModel(user, isTwoFactorEnabled, ciphers, billingInfo, billingHistoryInfo, _globalSettings, verifiedDomain, deviceVerificationRequired));
     }
 

src/Admin/Views/Users/Edit.cshtml

@@ -9,8 +9,7 @@
 
     var canViewUserInformation = AccessControlService.UserHasPermission(Permission.User_UserInformation_View);
     var canViewNewDeviceException = AccessControlService.UserHasPermission(Permission.User_NewDeviceException_Edit) &&
-                                    GlobalSettings.EnableNewDeviceVerification &&
-                                    FeatureService.IsEnabled(Bit.Core.FeatureFlagKeys.NewDeviceVerification);
+                                    GlobalSettings.EnableNewDeviceVerification;
     var canViewBillingInformation = AccessControlService.UserHasPermission(Permission.User_BillingInformation_View);
     var canViewGeneral = AccessControlService.UserHasPermission(Permission.User_GeneralDetails_View);
     var canViewPremium = AccessControlService.UserHasPermission(Permission.User_Premium_View);

src/Api/Utilities/CommandResultExtensions.cs

@@ -0,0 +1,31 @@
+using Bit.Core.Models.Commands;
+using Microsoft.AspNetCore.Mvc;
+
+namespace Bit.Api.Utilities;
+
+public static class CommandResultExtensions
+{
+    public static IActionResult MapToActionResult<T>(this CommandResult<T> commandResult)
+    {
+        return commandResult switch
+        {
+            NoRecordFoundFailure<T> failure => new ObjectResult(failure.ErrorMessages) { StatusCode = StatusCodes.Status404NotFound },
+            BadRequestFailure<T> failure => new ObjectResult(failure.ErrorMessages) { StatusCode = StatusCodes.Status400BadRequest },
+            Failure<T> failure => new ObjectResult(failure.ErrorMessages) { StatusCode = StatusCodes.Status400BadRequest },
+            Success<T> success => new ObjectResult(success.Data) { StatusCode = StatusCodes.Status200OK },
+            _ => throw new InvalidOperationException($"Unhandled commandResult type: {commandResult.GetType().Name}")
+        };
+    }
+
+    public static IActionResult MapToActionResult(this CommandResult commandResult)
+    {
+        return commandResult switch
+        {
+            NoRecordFoundFailure failure => new ObjectResult(failure.ErrorMessages) { StatusCode = StatusCodes.Status404NotFound },
+            BadRequestFailure failure => new ObjectResult(failure.ErrorMessages) { StatusCode = StatusCodes.Status400BadRequest },
+            Failure failure => new ObjectResult(failure.ErrorMessages) { StatusCode = StatusCodes.Status400BadRequest },
+            Success => new ObjectResult(new { }) { StatusCode = StatusCodes.Status200OK },
+            _ => throw new InvalidOperationException($"Unhandled commandResult type: {commandResult.GetType().Name}")
+        };
+    }
+}

src/Billing/Constants/BitPayInvoiceStatus.cs

@@ -0,0 +1,7 @@
+namespace Bit.Billing.Constants;
+
+public static class BitPayInvoiceStatus
+{
+    public const string Confirmed = "confirmed";
+    public const string Complete = "complete";
+}

src/Billing/Constants/BitPayNotificationCode.cs

@@ -0,0 +1,6 @@
+namespace Bit.Billing.Constants;
+
+public static class BitPayNotificationCode
+{
+    public const string InvoiceConfirmed = "invoice_confirmed";
+}

src/Billing/Controllers/BitPayController.cs

@@ -1,4 +1,5 @@
 using System.Globalization;
+using Bit.Billing.Constants;
 using Bit.Billing.Models;
 using Bit.Core.AdminConsole.Repositories;
 using Bit.Core.Billing.Services;
@@ -65,7 +66,7 @@ public class BitPayController : Controller
             return new BadRequestResult();
         }
 
-        if (model.Event.Name != "invoice_confirmed")
+        if (model.Event.Name != BitPayNotificationCode.InvoiceConfirmed)
         {
             // Only processing confirmed invoice events for now.
             return new OkResult();
@@ -75,20 +76,20 @@ public class BitPayController : Controller
         if (invoice == null)
         {
             // Request forged...?
-            _logger.LogWarning("Invoice not found. #" + model.Data.Id);
+            _logger.LogWarning("Invoice not found. #{InvoiceId}", model.Data.Id);
             return new BadRequestResult();
         }
 
-        if (invoice.Status != "confirmed" && invoice.Status != "completed")
+        if (invoice.Status != BitPayInvoiceStatus.Confirmed && invoice.Status != BitPayInvoiceStatus.Complete)
         {
-            _logger.LogWarning("Invoice status of '" + invoice.Status + "' is not acceptable. #" + invoice.Id);
+            _logger.LogWarning("Invoice status of '{InvoiceStatus}' is not acceptable. #{InvoiceId}", invoice.Status, invoice.Id);
             return new BadRequestResult();
         }
 
         if (invoice.Currency != "USD")
         {
             // Only process USD payments
-            _logger.LogWarning("Non USD payment received. #" + invoice.Id);
+            _logger.LogWarning("Non USD payment received. #{InvoiceId}", invoice.Id);
             return new OkResult();
         }
 

src/Billing/Services/Implementations/SubscriptionDeletedHandler.cs

@@ -41,10 +41,15 @@ public class SubscriptionDeletedHandler : ISubscriptionDeletedHandler
             return;
         }
 
-        if (organizationId.HasValue &&
-            subscription.CancellationDetails.Comment != providerMigrationCancellationComment &&
-            !subscription.CancellationDetails.Comment.Contains(addedToProviderCancellationComment))
+        if (organizationId.HasValue)
         {
+            if (!string.IsNullOrEmpty(subscription.CancellationDetails?.Comment) &&
+                (subscription.CancellationDetails.Comment == providerMigrationCancellationComment ||
+                 subscription.CancellationDetails.Comment.Contains(addedToProviderCancellationComment)))
+            {
+                return;
+            }
+
             await _organizationDisableCommand.DisableAsync(organizationId.Value, subscription.CurrentPeriodEnd);
         }
         else if (userId.HasValue)

src/Core/AdminConsole/Entities/Provider/Provider.cs

@@ -35,6 +35,7 @@ public class Provider : ITableObject<Guid>, ISubscriber
     public GatewayType? Gateway { get; set; }
     public string? GatewayCustomerId { get; set; }
     public string? GatewaySubscriptionId { get; set; }
+    public string? DiscountId { get; set; }
 
     public string? BillingEmailAddress() => BillingEmail?.ToLowerInvariant().Trim();
 

src/Core/AdminConsole/OrganizationFeatures/OrganizationUsers/UpdateOrganizationUserCommand.cs

@@ -117,7 +117,7 @@ public class UpdateOrganizationUserCommand : IUpdateOrganizationUserCommand
             throw new BadRequestException("Organization must have at least one confirmed owner.");
         }
 
-        if (collectionAccessList?.Count > 0)
+        if (collectionAccessList.Count > 0)
         {
             var invalidAssociations = collectionAccessList.Where(cas => cas.Manage && (cas.ReadOnly || cas.HidePasswords));
             if (invalidAssociations.Any())

src/Core/Billing/Constants/StripeConstants.cs

@@ -18,8 +18,15 @@ public static class StripeConstants
 
     public static class CouponIDs
     {
-        public const string MSPDiscount35 = "msp-discount-35";
+        public const string LegacyMSPDiscount = "msp-discount-35";
         public const string SecretsManagerStandalone = "sm-standalone";
+
+        public static class MSPDiscounts
+        {
+            public const string Open = "msp-open-discount";
+            public const string Silver = "msp-silver-discount";
+            public const string Gold = "msp-gold-discount";
+        }
     }
 
     public static class ErrorCodes

src/Core/Billing/Migration/Services/Implementations/ProviderMigrator.cs

@@ -254,7 +254,7 @@ public class ProviderMigrator(
 
             await stripeAdapter.CustomerUpdateAsync(customer.Id, new CustomerUpdateOptions
             {
-                Coupon = StripeConstants.CouponIDs.MSPDiscount35
+                Coupon = StripeConstants.CouponIDs.LegacyMSPDiscount
             });
 
             provider.GatewayCustomerId = customer.Id;

src/Core/Billing/Models/Sales/OrganizationSale.cs

@@ -46,7 +46,8 @@ public class OrganizationSale
         var customerSetup = new CustomerSetup
         {
             Coupon = signup.IsFromProvider
-            ? StripeConstants.CouponIDs.MSPDiscount35
+            // TODO: Remove when last of the legacy providers has been migrated.
+            ? StripeConstants.CouponIDs.LegacyMSPDiscount
             : signup.IsFromSecretsManagerTrial
                 ? StripeConstants.CouponIDs.SecretsManagerStandalone
                 : null

src/Core/Constants.cs

@@ -102,7 +102,6 @@ public static class AuthenticationSchemes
 public static class FeatureFlagKeys
 {
     /* Admin Console Team */
-    public const string ProviderClientVaultPrivacyBanner = "ac-2833-provider-client-vault-privacy-banner";
     public const string AccountDeprovisioning = "pm-10308-account-deprovisioning";
     public const string VerifiedSsoDomainEndpoint = "pm-12337-refactor-sso-details-endpoint";
     public const string DeviceApprovalRequestAdminNotifications = "pm-15637-device-approval-request-admin-notifications";
@@ -120,7 +119,6 @@ public static class FeatureFlagKeys
     public const string UseTreeWalkerApiForPageDetailsCollection = "use-tree-walker-api-for-page-details-collection";
     public const string DuoRedirect = "duo-redirect";
     public const string AC2101UpdateTrialInitiationEmail = "AC-2101-update-trial-initiation-email";
-    public const string AC1795_UpdatedSubscriptionStatusSection = "AC-1795_updated-subscription-status-section";
     public const string EmailVerification = "email-verification";
     public const string EmailVerificationDisableTimingDelays = "email-verification-disable-timing-delays";
     public const string ExtensionRefresh = "extension-refresh";

src/Core/Models/Commands/BadRequestFailure.cs

@@ -0,0 +1,23 @@
+namespace Bit.Core.Models.Commands;
+
+public class BadRequestFailure<T> : Failure<T>
+{
+    public BadRequestFailure(IEnumerable<string> errorMessage) : base(errorMessage)
+    {
+    }
+
+    public BadRequestFailure(string errorMessage) : base(errorMessage)
+    {
+    }
+}
+
+public class BadRequestFailure : Failure
+{
+    public BadRequestFailure(IEnumerable<string> errorMessage) : base(errorMessage)
+    {
+    }
+
+    public BadRequestFailure(string errorMessage) : base(errorMessage)
+    {
+    }
+}

src/Core/Models/Commands/CommandResult.cs

@@ -1,4 +1,6 @@
-namespace Bit.Core.Models.Commands;
+#nullable enable
+
+namespace Bit.Core.Models.Commands;
 
 public class CommandResult(IEnumerable<string> errors)
 {
@@ -10,3 +12,39 @@ public class CommandResult(IEnumerable<string> errors)
 
     public CommandResult() : this(Array.Empty<string>()) { }
 }
+
+public class Failure : CommandResult
+{
+    protected Failure(IEnumerable<string> errorMessages) : base(errorMessages)
+    {
+
+    }
+    public Failure(string errorMessage) : base(errorMessage)
+    {
+
+    }
+}
+
+public class Success : CommandResult
+{
+}
+
+public abstract class CommandResult<T>
+{
+
+}
+
+public class Success<T>(T data) : CommandResult<T>
+{
+    public T? Data { get; init; } = data;
+}
+
+public class Failure<T>(IEnumerable<string> errorMessage) : CommandResult<T>
+{
+    public IEnumerable<string> ErrorMessages { get; init; } = errorMessage;
+
+    public Failure(string errorMessage) : this(new[] { errorMessage })
+    {
+    }
+}
+

src/Core/Models/Commands/NoRecordFoundFailure.cs

@@ -0,0 +1,24 @@
+namespace Bit.Core.Models.Commands;
+
+public class NoRecordFoundFailure<T> : Failure<T>
+{
+    public NoRecordFoundFailure(IEnumerable<string> errorMessage) : base(errorMessage)
+    {
+    }
+
+    public NoRecordFoundFailure(string errorMessage) : base(errorMessage)
+    {
+    }
+}
+
+public class NoRecordFoundFailure : Failure
+{
+    public NoRecordFoundFailure(IEnumerable<string> errorMessage) : base(errorMessage)
+    {
+    }
+
+    public NoRecordFoundFailure(string errorMessage) : base(errorMessage)
+    {
+    }
+}
+

src/Core/Services/Implementations/StripePaymentService.cs

@@ -1609,15 +1609,12 @@ public class StripePaymentService : IPaymentService
         {
             subscriptionInfo.Subscription = new SubscriptionInfo.BillingSubscription(sub);
 
-            if (_featureService.IsEnabled(FeatureFlagKeys.AC1795_UpdatedSubscriptionStatusSection))
-            {
-                var (suspensionDate, unpaidPeriodEndDate) = await GetSuspensionDateAsync(sub);
+            var (suspensionDate, unpaidPeriodEndDate) = await GetSuspensionDateAsync(sub);
 
-                if (suspensionDate.HasValue && unpaidPeriodEndDate.HasValue)
-                {
-                    subscriptionInfo.Subscription.SuspensionDate = suspensionDate;
-                    subscriptionInfo.Subscription.UnpaidPeriodEndDate = unpaidPeriodEndDate;
-                }
+            if (suspensionDate.HasValue && unpaidPeriodEndDate.HasValue)
+            {
+                subscriptionInfo.Subscription.SuspensionDate = suspensionDate;
+                subscriptionInfo.Subscription.UnpaidPeriodEndDate = unpaidPeriodEndDate;
             }
         }
 

src/Sql/dbo/Stored Procedures/Provider_Create.sql

@@ -17,7 +17,8 @@
     @RevisionDate DATETIME2(7),
     @Gateway TINYINT = 0,
     @GatewayCustomerId VARCHAR(50) = NULL,
-    @GatewaySubscriptionId VARCHAR(50) = NULL
+    @GatewaySubscriptionId VARCHAR(50) = NULL,
+    @DiscountId VARCHAR(50) = NULL
 AS
 BEGIN
     SET NOCOUNT ON
@@ -42,7 +43,8 @@ BEGIN
         [RevisionDate],
         [Gateway],
         [GatewayCustomerId],
-        [GatewaySubscriptionId]
+        [GatewaySubscriptionId],
+        [DiscountId]
     )
     VALUES
     (
@@ -64,6 +66,7 @@ BEGIN
         @RevisionDate,
         @Gateway,
         @GatewayCustomerId,
-        @GatewaySubscriptionId
+        @GatewaySubscriptionId,
+        @DiscountId
     )
 END

src/Sql/dbo/Stored Procedures/Provider_Update.sql

@@ -17,7 +17,8 @@
     @RevisionDate DATETIME2(7),
     @Gateway TINYINT = 0,
     @GatewayCustomerId VARCHAR(50) = NULL,
-    @GatewaySubscriptionId VARCHAR(50) = NULL
+    @GatewaySubscriptionId VARCHAR(50) = NULL,
+    @DiscountId VARCHAR(50) = NULL
 AS
 BEGIN
     SET NOCOUNT ON
@@ -42,7 +43,8 @@ BEGIN
         [RevisionDate] = @RevisionDate,
         [Gateway] = @Gateway,
         [GatewayCustomerId] = @GatewayCustomerId,
-        [GatewaySubscriptionId] = @GatewaySubscriptionId
+        [GatewaySubscriptionId] = @GatewaySubscriptionId,
+        [DiscountId] = @DiscountId
     WHERE
         [Id] = @Id
 END

src/Sql/dbo/Tables/Provider.sql

@@ -18,5 +18,6 @@
     [Gateway]               TINYINT          NULL,
     [GatewayCustomerId]     VARCHAR (50)     NULL,
     [GatewaySubscriptionId] VARCHAR (50)     NULL,
+    [DiscountId]            VARCHAR (50)     NULL,
     CONSTRAINT [PK_Provider] PRIMARY KEY CLUSTERED ([Id] ASC)
 );

test/Api.Test/Utilities/CommandResultExtensionTests.cs

@@ -0,0 +1,107 @@
+using Bit.Api.Utilities;
+using Bit.Core.Models.Commands;
+using Bit.Core.Vault.Entities;
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Mvc;
+using Xunit;
+
+namespace Bit.Api.Test.Utilities;
+
+public class CommandResultExtensionTests
+{
+    public static IEnumerable<object[]> WithGenericTypeTestCases()
+    {
+        yield return new object[]
+        {
+            new NoRecordFoundFailure<Cipher>(new[] { "Error 1", "Error 2" }),
+            new ObjectResult(new[] { "Error 1", "Error 2" }) { StatusCode = StatusCodes.Status404NotFound }
+        };
+        yield return new object[]
+        {
+            new BadRequestFailure<Cipher>("Error 3"),
+            new ObjectResult(new[] { "Error 3" }) { StatusCode = StatusCodes.Status400BadRequest }
+        };
+        yield return new object[]
+        {
+            new Failure<Cipher>("Error 4"),
+            new ObjectResult(new[] { "Error 4" }) { StatusCode = StatusCodes.Status400BadRequest }
+        };
+        var cipher = new Cipher() { Id = Guid.NewGuid() };
+
+        yield return new object[]
+        {
+            new Success<Cipher>(cipher),
+            new ObjectResult(cipher) { StatusCode = StatusCodes.Status200OK }
+        };
+    }
+
+
+    [Theory]
+    [MemberData(nameof(WithGenericTypeTestCases))]
+    public void MapToActionResult_WithGenericType_ShouldMapToHttpResponse(CommandResult<Cipher> input, ObjectResult expected)
+    {
+        var result = input.MapToActionResult();
+
+        Assert.Equivalent(expected, result);
+    }
+
+
+    [Fact]
+    public void MapToActionResult_WithGenericType_ShouldThrowExceptionForUnhandledCommandResult()
+    {
+        var result = new NotImplementedCommandResult();
+
+        Assert.Throws<InvalidOperationException>(() => result.MapToActionResult());
+    }
+
+    public static IEnumerable<object[]> TestCases()
+    {
+        yield return new object[]
+        {
+            new NoRecordFoundFailure(new[] { "Error 1", "Error 2" }),
+            new ObjectResult(new[] { "Error 1", "Error 2" }) { StatusCode = StatusCodes.Status404NotFound }
+        };
+        yield return new object[]
+        {
+            new BadRequestFailure("Error 3"),
+            new ObjectResult(new[] { "Error 3" }) { StatusCode = StatusCodes.Status400BadRequest }
+        };
+        yield return new object[]
+        {
+            new Failure("Error 4"),
+            new ObjectResult(new[] { "Error 4" }) { StatusCode = StatusCodes.Status400BadRequest }
+        };
+        yield return new object[]
+        {
+            new Success(),
+            new ObjectResult(new { }) { StatusCode = StatusCodes.Status200OK }
+        };
+    }
+
+    [Theory]
+    [MemberData(nameof(TestCases))]
+    public void MapToActionResult_ShouldMapToHttpResponse(CommandResult input, ObjectResult expected)
+    {
+        var result = input.MapToActionResult();
+
+        Assert.Equivalent(expected, result);
+    }
+
+    [Fact]
+    public void MapToActionResult_ShouldThrowExceptionForUnhandledCommandResult()
+    {
+        var result = new NotImplementedCommandResult<Cipher>();
+
+        Assert.Throws<InvalidOperationException>(() => result.MapToActionResult());
+    }
+}
+
+public class NotImplementedCommandResult<T> : CommandResult<T>
+{
+
+}
+
+public class NotImplementedCommandResult : CommandResult
+{
+
+}

util/Migrator/DbScripts/2025-02-11_00_AddColumn_ProviderDiscountId.sql

@@ -0,0 +1,171 @@
+-- Add 'DiscountId' column to 'Provider' table.
+IF COL_LENGTH('[dbo].[Provider]', 'DiscountId') IS NULL
+    BEGIN
+        ALTER TABLE
+            [dbo].[Provider]
+            ADD
+                [DiscountId] VARCHAR(50) NULL;
+    END
+GO
+
+-- Recreate 'ProviderView' so that it includes the 'DiscountId' column.
+CREATE OR ALTER VIEW [dbo].[ProviderView]
+AS
+SELECT
+    *
+FROM
+    [dbo].[Provider]
+GO
+
+-- Alter 'Provider_Create' SPROC to add 'DiscountId' column.
+CREATE OR ALTER PROCEDURE [dbo].[Provider_Create]
+    @Id UNIQUEIDENTIFIER OUTPUT,
+    @Name NVARCHAR(50),
+    @BusinessName NVARCHAR(50),
+    @BusinessAddress1 NVARCHAR(50),
+    @BusinessAddress2 NVARCHAR(50),
+    @BusinessAddress3 NVARCHAR(50),
+    @BusinessCountry VARCHAR(2),
+    @BusinessTaxNumber NVARCHAR(30),
+    @BillingEmail NVARCHAR(256),
+    @BillingPhone NVARCHAR(50) = NULL,
+    @Status TINYINT,
+    @Type TINYINT = 0,
+    @UseEvents BIT,
+    @Enabled BIT,
+    @CreationDate DATETIME2(7),
+    @RevisionDate DATETIME2(7),
+    @Gateway TINYINT = 0,
+    @GatewayCustomerId VARCHAR(50) = NULL,
+    @GatewaySubscriptionId VARCHAR(50) = NULL,
+    @DiscountId VARCHAR(50) = NULL
+AS
+BEGIN
+    SET NOCOUNT ON
+
+    INSERT INTO [dbo].[Provider]
+    (
+        [Id],
+        [Name],
+        [BusinessName],
+        [BusinessAddress1],
+        [BusinessAddress2],
+        [BusinessAddress3],
+        [BusinessCountry],
+        [BusinessTaxNumber],
+        [BillingEmail],
+        [BillingPhone],
+        [Status],
+        [Type],
+        [UseEvents],
+        [Enabled],
+        [CreationDate],
+        [RevisionDate],
+        [Gateway],
+        [GatewayCustomerId],
+        [GatewaySubscriptionId],
+        [DiscountId]
+    )
+    VALUES
+        (
+         @Id,
+         @Name,
+         @BusinessName,
+         @BusinessAddress1,
+         @BusinessAddress2,
+         @BusinessAddress3,
+         @BusinessCountry,
+         @BusinessTaxNumber,
+         @BillingEmail,
+         @BillingPhone,
+         @Status,
+         @Type,
+         @UseEvents,
+         @Enabled,
+         @CreationDate,
+         @RevisionDate,
+         @Gateway,
+         @GatewayCustomerId,
+         @GatewaySubscriptionId,
+         @DiscountId
+        )
+END
+GO
+
+-- Alter 'Provider_Update' SPROC to add 'DiscountId' column.
+CREATE OR ALTER PROCEDURE [dbo].[Provider_Update]
+    @Id UNIQUEIDENTIFIER,
+    @Name NVARCHAR(50),
+    @BusinessName NVARCHAR(50),
+    @BusinessAddress1 NVARCHAR(50),
+    @BusinessAddress2 NVARCHAR(50),
+    @BusinessAddress3 NVARCHAR(50),
+    @BusinessCountry VARCHAR(2),
+    @BusinessTaxNumber NVARCHAR(30),
+    @BillingEmail NVARCHAR(256),
+    @BillingPhone NVARCHAR(50) = NULL,
+    @Status TINYINT,
+    @Type TINYINT = 0,
+    @UseEvents BIT,
+    @Enabled BIT,
+    @CreationDate DATETIME2(7),
+    @RevisionDate DATETIME2(7),
+    @Gateway TINYINT = 0,
+    @GatewayCustomerId VARCHAR(50) = NULL,
+    @GatewaySubscriptionId VARCHAR(50) = NULL,
+    @DiscountId VARCHAR(50) = NULL
+AS
+BEGIN
+    SET NOCOUNT ON
+
+    UPDATE
+        [dbo].[Provider]
+    SET
+        [Name] = @Name,
+        [BusinessName] = @BusinessName,
+        [BusinessAddress1] = @BusinessAddress1,
+        [BusinessAddress2] = @BusinessAddress2,
+        [BusinessAddress3] = @BusinessAddress3,
+        [BusinessCountry] = @BusinessCountry,
+        [BusinessTaxNumber] = @BusinessTaxNumber,
+        [BillingEmail] = @BillingEmail,
+        [BillingPhone] = @BillingPhone,
+        [Status] = @Status,
+        [Type] = @Type,
+        [UseEvents] = @UseEvents,
+        [Enabled] = @Enabled,
+        [CreationDate] = @CreationDate,
+        [RevisionDate] = @RevisionDate,
+        [Gateway] = @Gateway,
+        [GatewayCustomerId] = @GatewayCustomerId,
+        [GatewaySubscriptionId] = @GatewaySubscriptionId,
+        [DiscountId] = @DiscountId
+    WHERE
+        [Id] = @Id
+END
+GO
+
+-- Refresh modules for SPROCs reliant on 'Provider' table/view.
+IF OBJECT_ID('[dbo].[Provider_ReadAbilities]') IS NOT NULL
+    BEGIN
+        EXECUTE sp_refreshsqlmodule N'[dbo].[Provider_ReadAbilities]';
+    END
+GO
+
+IF OBJECT_ID('[dbo].[Provider_ReadById]') IS NOT NULL
+    BEGIN
+        EXECUTE sp_refreshsqlmodule N'[dbo].[Provider_ReadById]';
+    END
+GO
+
+IF OBJECT_ID('[dbo].[Provider_ReadByOrganizationId]') IS NOT NULL
+    BEGIN
+        EXECUTE sp_refreshsqlmodule N'[dbo].[Provider_ReadByOrganizationId]';
+    END
+GO
+
+IF OBJECT_ID('[dbo].[Provider_Search]') IS NOT NULL
+    BEGIN
+        EXECUTE sp_refreshsqlmodule N'[dbo].[Provider_Search]';
+    END
+GO

util/MySqlMigrations/Migrations/20250213140357_AddColumn_ProviderDiscountId.cs

@@ -0,0 +1,28 @@
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace Bit.MySqlMigrations.Migrations;
+
+/// <inheritdoc />
+public partial class AddColumn_ProviderDiscountId : Migration
+{
+    /// <inheritdoc />
+    protected override void Up(MigrationBuilder migrationBuilder)
+    {
+        migrationBuilder.AddColumn<string>(
+            name: "DiscountId",
+            table: "Provider",
+            type: "longtext",
+            nullable: true)
+            .Annotation("MySql:CharSet", "utf8mb4");
+    }
+
+    /// <inheritdoc />
+    protected override void Down(MigrationBuilder migrationBuilder)
+    {
+        migrationBuilder.DropColumn(
+            name: "DiscountId",
+            table: "Provider");
+    }
+}

util/MySqlMigrations/Migrations/DatabaseContextModelSnapshot.cs

@@ -284,6 +284,9 @@ namespace Bit.MySqlMigrations.Migrations
                     b.Property<DateTime>("CreationDate")
                         .HasColumnType("datetime(6)");
 
+                    b.Property<string>("DiscountId")
+                        .HasColumnType("longtext");
+
                     b.Property<bool>("Enabled")
                         .HasColumnType("tinyint(1)");
 

util/PostgresMigrations/Migrations/20250213140406_AddColumn_ProviderDiscountId.cs

@@ -0,0 +1,27 @@
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace Bit.PostgresMigrations.Migrations;
+
+/// <inheritdoc />
+public partial class AddColumn_ProviderDiscountId : Migration
+{
+    /// <inheritdoc />
+    protected override void Up(MigrationBuilder migrationBuilder)
+    {
+        migrationBuilder.AddColumn<string>(
+            name: "DiscountId",
+            table: "Provider",
+            type: "text",
+            nullable: true);
+    }
+
+    /// <inheritdoc />
+    protected override void Down(MigrationBuilder migrationBuilder)
+    {
+        migrationBuilder.DropColumn(
+            name: "DiscountId",
+            table: "Provider");
+    }
+}

util/PostgresMigrations/Migrations/DatabaseContextModelSnapshot.cs

@@ -287,6 +287,9 @@ namespace Bit.PostgresMigrations.Migrations
                     b.Property<DateTime>("CreationDate")
                         .HasColumnType("timestamp with time zone");
 
+                    b.Property<string>("DiscountId")
+                        .HasColumnType("text");
+
                     b.Property<bool>("Enabled")
                         .HasColumnType("boolean");
 

util/SqliteMigrations/Migrations/20250213140401_AddColumn_ProviderDiscountId.cs

@@ -0,0 +1,27 @@
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace Bit.SqliteMigrations.Migrations;
+
+/// <inheritdoc />
+public partial class AddColumn_ProviderDiscountId : Migration
+{
+    /// <inheritdoc />
+    protected override void Up(MigrationBuilder migrationBuilder)
+    {
+        migrationBuilder.AddColumn<string>(
+            name: "DiscountId",
+            table: "Provider",
+            type: "TEXT",
+            nullable: true);
+    }
+
+    /// <inheritdoc />
+    protected override void Down(MigrationBuilder migrationBuilder)
+    {
+        migrationBuilder.DropColumn(
+            name: "DiscountId",
+            table: "Provider");
+    }
+}

util/SqliteMigrations/Migrations/DatabaseContextModelSnapshot.cs

@@ -279,6 +279,9 @@ namespace Bit.SqliteMigrations.Migrations
                     b.Property<DateTime>("CreationDate")
                         .HasColumnType("TEXT");
 
+                    b.Property<string>("DiscountId")
+                        .HasColumnType("TEXT");
+
                     b.Property<bool>("Enabled")
                         .HasColumnType("INTEGER");