Fixed an issue with the "OAuth" substring being removed from the "oauth_body_hash" values

Author: jaaufauvre

Mastercard.Developer.OAuth1Signer.Core/Signers/NetHttpClientSigner.cs

@@ -47,7 +47,8 @@ private async Task SignPrivateAsync(HttpRequestMessage request)
             // Generate the header and add it to the request
             var methodString = request.Method.ToString();
             var header = OAuth.GetAuthorizationHeader(request.RequestUri.AbsoluteUri, methodString, payload, Encoding, ConsumerKey, SigningKey);
-            request.Headers.Authorization = new AuthenticationHeaderValue("OAuth", header.Replace("OAuth", string.Empty));
+            var parameter = header.Substring(6, header.Length - 6); // Remove the "OAuth" scheme
+            request.Headers.Authorization = new AuthenticationHeaderValue("OAuth", parameter);
         }
     }
 }

Mastercard.Developer.OAuth1Signer.Tests/NetCore2/Signers/NetHttpClientSignerTest.cs

@@ -114,5 +114,24 @@ public void TestSignAsync_ShouldThrowArgumentNullException_WhenNullRequest()
         }
 
         #endregion
+
+        [TestMethod]
+        public void TestSign_ShouldNotRemoveOAuthSubstringFromBodyDigests()
+        {
+            // GIVEN
+            var request = new HttpRequestMessage
+            {
+                Method = HttpMethod.Post,
+                RequestUri = new Uri("https://api.mastercard.com/service"),
+                Content = new StringContent("{ \"uuid\": \"e8b57e13-a771-4d49-9419-4d1168b0a684\" }") // The SHA-256 digest is "ugxcxkGBiQwu5kGnpvKgrbkOAuthLE+qtCXBDeZ/D/I="
+            };
+
+            // WHEN
+            CreateHttpClientSigner().Sign(request);
+
+            // THEN
+            Assert.AreEqual("OAuth", request.Headers.Authorization.Scheme);
+            Assert.IsTrue(request.Headers.Authorization.Parameter.Contains("bkOAuthLE"));
+        }
     }
 }