Merge pull request #1129 from andersjonsson/develop

fixed an error with reading Base64-encoded data and added a test

Author: andersjonsson

src/SoapCore.Tests/XDocumentXmlReaderTests.cs

@@ -0,0 +1,52 @@
+using System;
+using System.Collections.Generic;
+using System.IO;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using System.Xml.Linq;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+
+namespace SoapCore.Tests
+{
+	[TestClass]
+	public class XDocumentXmlReaderTests
+	{
+		[TestMethod]
+		public void TestReadBase64()
+		{
+			var xml = @"
+<Request>
+	<Base64Data>
+QkVHSU4KQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhQmxhYmxhYmxhbGJsYUJsYWJsYWJsYWxibGFCbGFibGFibGFsYmxhCkVORA==
+	</Base64Data>
+</Request>
+";
+			var xdoc = XDocument.Parse(xml);
+			var reader = new XDocumentXmlReader(xdoc);
+
+			reader.Read();
+			reader.Read();
+			reader.Read();
+
+			var buffer = new byte[1024];
+			var ms = new MemoryStream();
+			var start = 0;
+			while (true)
+			{
+				var i = reader.ReadContentAsBase64(buffer, start, 1024);
+				ms.Write(buffer, 0, i);
+				start += i;
+
+				if (i < buffer.Length)
+				{
+					break;
+				}
+			}
+
+			ms.Position = 0;
+
+			Assert.AreEqual(1336, ms.Length);
+		}
+	}
+}

src/SoapCore/SoapCore.csproj

@@ -2,7 +2,7 @@
 
 	<PropertyGroup>
 		<Description>SOAP protocol middleware for ASP.NET Core</Description>
-		<Version>1.2.1.2</Version>
+		<Version>1.2.1.3</Version>
 		<Authors>Digital Design</Authors>
 		<TargetFrameworks>netstandard2.0;netstandard2.1;net8.0;netcoreapp3.1;</TargetFrameworks>
 		<PackageId>SoapCore</PackageId>

src/SoapCore/XDocumentXmlReader.cs

@@ -37,7 +37,8 @@ public XDocumentXmlReader(XDocument document)
 		public override int ReadContentAsBase64(byte[] buffer, int index, int count)
 		{
 			EnsureBinaryStream(true);
-			return _binaryStream!.Read(buffer, index, count);
+
+			return _binaryStream!.Read(buffer, 0, count);
 		}
 
 		public override int ReadElementContentAsBase64(byte[] buffer, int index, int count)
@@ -53,7 +54,8 @@ public override int ReadElementContentAsBase64(byte[] buffer, int index, int cou
 		public override int ReadContentAsBinHex(byte[] buffer, int index, int count)
 		{
 			EnsureBinaryStream(false);
-			return _binaryStream!.Read(buffer, index, count);
+
+			return _binaryStream!.Read(buffer, 0, count);
 		}
 
 		public override int ReadElementContentAsBinHex(byte[] buffer, int index, int count)