SP-1149: use resource token in webhook resend requests

.github/workflows/build.yml

@@ -6,6 +6,7 @@ jobs:
   build-and-test:
     runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
         dotnet-version: [ '6.0', '7.0', '8.0' ]
 
@@ -16,7 +17,7 @@ jobs:
         with:
           dotnet-version: ${{ matrix.dotnet-version }}
       - name: Install dependencies
-        run: dotnet restore
+        run: dotnet restore -p:TargetFramework=net${{ matrix.dotnet-version }}
       - name: Build
         run: dotnet build --no-restore -f net${{ matrix.dotnet-version }}
       - name: Test

BitPay/Client.cs

@@ -316,11 +316,15 @@ public async Task<Invoice> CancelInvoiceByGuid(string refundGuid)
         ///     Request a webhook to be resent.
         /// </summary>
         /// <param name="invoiceId">Invoice ID.</param>
+        /// <param name="invoiceToken">
+        ///    The resource token for the invoiceId. 
+        ///    This token can be retrieved from the Bitpay's invoice object.
+        /// </param>
         /// <returns>Status of request</returns>
         /// <exception cref="ArgumentNullException"></exception>
-        public async Task<Boolean> RequestInvoiceWebhookToBeResent(string invoiceId)
+        public async Task<Boolean> RequestInvoiceWebhookToBeResent(string invoiceId, string invoiceToken)
         {
-            return await CreateInvoiceClient().RequestInvoiceWebhookToBeResent(invoiceId).ConfigureAwait(false);
+            return await CreateInvoiceClient().RequestInvoiceWebhookToBeResent(invoiceId, invoiceToken).ConfigureAwait(false);
         }
 
         /// <summary>
@@ -420,12 +424,16 @@ public async Task<Refund> UpdateRefundByGuid(string refundGuid, string status)
         ///     Send a refund notification
         /// </summary>
         /// <param name="refundId">A BitPay refundId </param>
+        /// <param name="refundToken">
+        ///    The resource token for the refundId. 
+        ///    This token can be retrieved from the Bitpay's refund object.
+        /// </param>
         /// <returns>An updated Refund Object </returns>
         /// <throws>RefundCreationException RefundCreationException class </throws>
         /// <throws>BitPayException BitPayException class </throws>
-        public async Task<Boolean> SendRefundNotification(string refundId)
+        public async Task<Boolean> SendRefundNotification(string refundId, string refundToken)
         {
-            return await CreateRefundClient().SendRefundNotification(refundId).ConfigureAwait(false);
+            return await CreateRefundClient().SendRefundNotification(refundId, refundToken).ConfigureAwait(false);
         }
 
         /// <summary>

BitPay/Clients/InvoiceClient.cs

@@ -310,13 +310,17 @@ public async Task<Invoice> CancelInvoiceByGuid(string guidId, bool forceCancel =
         ///     Request a webhook to be resent.
         /// </summary>
         /// <param name="invoiceId">Invoice ID.</param>
+        /// <param name="invoiceToken">
+        ///    The resource token for the invoiceId. 
+        ///    This token can be retrieved from the Bitpay's invoice object.
+        /// </param>
         /// <returns>Status of request</returns>
         /// <exception cref="BitPayGenericException">BitPayGenericException class</exception>
         /// <exception cref="BitPayApiException">BitPayApiException class</exception>
-        public async Task<bool> RequestInvoiceWebhookToBeResent(string invoiceId)
+        public async Task<bool> RequestInvoiceWebhookToBeResent(string invoiceId, string invoiceToken)
         {
             var parameters = ResourceClientUtil.InitParams();
-            parameters.Add("token", _accessTokens.GetAccessToken((Facade.Merchant)));
+            parameters.Add("token", invoiceToken);
 
             string json = null!;
 
@@ -407,4 +411,4 @@ public async Task<Invoice> PayInvoice(string invoiceId, string status)
             return invoice;
         }
     }
-}
+}

BitPay/Clients/RefundClient.cs

@@ -302,13 +302,17 @@ public async Task<Refund> CancelByGuid(string refundGuid)
         ///     Send a refund notification
         /// </summary>
         /// <param name="refundId">A BitPay refundId </param>
+        /// <param name="refundToken">
+        ///    The resource token for the refundId. 
+        ///    This token can be retrieved from the Bitpay's refund object.
+        /// </param>
         /// <returns>An updated Refund Object </returns>
         /// <exception cref="BitPayGenericException">BitPayGenericException class</exception>
         /// <exception cref="BitPayApiException">BitPayApiException class</exception>
-        public async Task<bool> SendRefundNotification(string refundId)
+        public async Task<bool> SendRefundNotification(string refundId, string refundToken)
         {
             var parameters = ResourceClientUtil.InitParams();
-            parameters.Add("token", _accessTokens.GetAccessToken(Facade.Merchant));
+            parameters.Add("token", refundToken);
 
             string json;
 
@@ -342,4 +346,4 @@ public async Task<bool> SendRefundNotification(string refundId)
             }
         }
     }
-}
+}

BitPay/Exceptions/BitPayApiException.cs

@@ -30,5 +30,9 @@ public string? Code
                 return _code;
             }
         }
+
+        protected BitPayApiException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
     }
-}
+}

BitPay/Exceptions/BitPayException.cs

@@ -15,5 +15,9 @@
         public BitPayException(string message) : base(message)
         {
         }
+
+        protected BitPayException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
     }
-}
+}

BitPay/Exceptions/BitPayGenericException.cs

@@ -15,5 +15,9 @@ public class BitPayGenericException : BitPayException
         public BitPayGenericException(string message) : base(message)
         {
         }
+
+        protected BitPayGenericException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
     }
-}
+}

BitPay/Exceptions/BitPayValidationException.cs

@@ -15,5 +15,9 @@ public class BitPayValidationException : BitPayGenericException
         public BitPayValidationException(string message) : base(message)
         {
         }
+
+        protected BitPayValidationException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
     }
-}
+}

BitPay/Models/Invoice/RecipientStatus.cs

@@ -3,7 +3,7 @@
 
 namespace BitPay.Models.Invoice
 {
-    public class RecipientStatus
+    public static class RecipientStatus
     {
         public const string Active = "active";
         public const string Invited = "invited";
@@ -12,4 +12,4 @@ public class RecipientStatus
         public const string Unverified = "unverified";
         public const string Verified = "verified";
     }
-}
+}

BitPay/Models/Invoice/Refund.cs

@@ -27,6 +27,9 @@ public class Refund
         [JsonProperty(PropertyName = "guid")]
         public string? ResourceGuid { get; set; }
 
+        [JsonProperty(PropertyName = "token")] 
+        public string? Token { get; set; }
+
         [JsonProperty(PropertyName = "refundAddress")]
         public string? RefundAddress { get; set; }
 

BitPayFunctionalTest/BitPayFunctionalTest.cs

@@ -146,6 +146,7 @@ public async Task it_should_test_refunds_requests()
             var refundToCreateRequest = new Refund(invoiceId: invoiceId, amount: 10.0M);
             var refund = await _client.CreateRefund(refundToCreateRequest);
             var refundId = refund.Id!;
+            var refundToken = refund.Token!;
 
             var retrieveRefund = await _client.GetRefund(refundId);
             Assert.Equal(refundId, retrieveRefund.Id);
@@ -158,7 +159,7 @@ public async Task it_should_test_refunds_requests()
             Assert.NotEmpty(retrieveRefundByInvoiceId);
             retrieveRefundByInvoiceId.Exists(refundByInvoice => refundByInvoice.Invoice == invoiceId);
 
-            var refundNotification = await _client.SendRefundNotification(refundId);
+            var refundNotification = await _client.SendRefundNotification(refundId, refundToken);
             Assert.True(refundNotification);
 
             var cancelRefund = await _client.CancelRefund(refundId);
@@ -480,4 +481,4 @@ private static string GetBitPayUnitTestPath()
             return bitPayUnitTestPath;
         }
     }
-}
+}

BitPayUnitTest/ClientTest.cs

@@ -7,13 +7,16 @@
 using BitPay;
 using BitPay.Clients;
 using BitPay.Exceptions;
+using BitPay.Models;
 using BitPay.Models.Bill;
 using BitPay.Models.Invoice;
 using BitPay.Models.Payout;
 using BitPay.Utils;
 
 using Moq;
 
+using Newtonsoft.Json;
+
 using Environment = BitPay.Environment;
 using SystemEnvironment = System.Environment;
 
@@ -826,11 +829,15 @@ public void it_should_cancel_invoice_by_guid()
         [Fact]
         public void it_should_request_invoice_webhook_to_be_resent()
         {
+            var invoiceToken = "cM78LHk17Q8fktDE6QLBBFfvH1QKBhRkHibTLcxhgzsu3VDRvSyu3CGi17DuwYxhT";var responseObject = new 
+            {
+                token = invoiceToken
+            };
             // given
             HttpContent response = new StringContent(File.ReadAllText(GetJsonResponsePath() + "invoiceWebhookResponse.json"));
             _bitPayClient.Setup(b => b.Post(
                 "invoices/Hpqc63wvE1ZjzeeH4kEycF/notifications",
-            "{\"token\":\"merchantToken\"}",
+                JsonConvert.SerializeObject(responseObject),
                 false
             )).ReturnsAsync(new HttpResponseMessage
             {
@@ -841,7 +848,7 @@ public void it_should_request_invoice_webhook_to_be_resent()
             });
 
             // when
-            var result = GetTestedClassAsMerchant().RequestInvoiceWebhookToBeResent("Hpqc63wvE1ZjzeeH4kEycF").Result;
+            var result = GetTestedClassAsMerchant().RequestInvoiceWebhookToBeResent("Hpqc63wvE1ZjzeeH4kEycF", invoiceToken).Result;
 
             // then
             Assert.True(result);
@@ -1665,11 +1672,16 @@ public void it_should_update_refund_by_guid()
         [Fact]
         public void it_should_send_refund_notification()
         {
+            var refundToken = "cM78LHk17Q8fktDE6QLBBFfvH1QKBhRkHibTLcxhgzsu3VDRvSyu3CGi17DuwYxhT";
+            var responseObject = new 
+            {
+                token = refundToken
+            };
             // given
             HttpContent response = new StringContent(File.ReadAllText(GetJsonResponsePath() + "sendRefundNotificationResponse.json"));
             _bitPayClient.Setup(b => b.Post(
                 "refunds/WoE46gSLkJQS48RJEiNw3L/notifications",
-                "{\"token\":\"merchantToken\"}",
+                JsonConvert.SerializeObject(responseObject),
                 true
             )).ReturnsAsync(new HttpResponseMessage
             {
@@ -1679,7 +1691,7 @@ public void it_should_send_refund_notification()
             });
 
             // when
-            var result = GetTestedClassAsMerchant().SendRefundNotification("WoE46gSLkJQS48RJEiNw3L").Result;
+            var result = GetTestedClassAsMerchant().SendRefundNotification("WoE46gSLkJQS48RJEiNw3L", refundToken).Result;
 
             // then
             Assert.True(result);