Merge pull request #5 from PSWinCom/receiver-bad-reply

Make GatewayReceiver write correct response body

Author: fmelink

PSWin.Client/LinkMobility.PSWin.Client.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>netstandard2.0</TargetFramework>
     <GeneratePackageOnBuild>True</GeneratePackageOnBuild>
-    <Version>1.2.0</Version>
+    <Version>1.3.0</Version>
     <Authors>Link Mobility NE</Authors>
     <Company></Company>
     <Description>Send SMS through PSWin gateway.</Description>

PSWin.Receiver/GatewayReceiver.cs

@@ -5,7 +5,6 @@
 using System;
 using System.IO;
 using System.Net;
-using System.Net.Http;
 using System.Threading.Tasks;
 using System.Xml;
 using System.Xml.Linq;
@@ -17,6 +16,7 @@ public class GatewayReceiver
         public delegate Task MoReceiver(MoMessage message);
         public delegate Task DrReceiver(DrMessage message);
 
+        private const string XmlOkResponse = "<?xml version=\"1.0\"?><MSGLST><MSG><ID>1</ID><STATUS>OK</STATUS></MSG></MSGLST>";
         private readonly MoReceiver moReceiver;
         private readonly DrReceiver drReceiver;
 
@@ -31,7 +31,7 @@ public async Task ReceiveMobileOriginatedMessageAsync(HttpContext context)
             var body = await new StreamReader(context.Request.Body).ReadToEndAsync();
             var result = await ReceiveMobileOriginatedMessageAsync(body);
             context.Response.StatusCode = (int)result.status;
-            await new StreamWriter(context.Response.Body).WriteAsync(result.responseBody);
+            await HttpResponseWritingExtensions.WriteAsync(context.Response, result.responseBody);
         }
 
         public async Task<(HttpStatusCode status, string responseBody)> ReceiveMobileOriginatedMessageAsync(string requestBody)
@@ -46,7 +46,7 @@ public async Task ReceiveMobileOriginatedMessageAsync(HttpContext context)
             {
                 var momessage = MoParser.Parse(document);
                 await moReceiver.Invoke(momessage);
-                return (HttpStatusCode.OK, string.Empty);
+                return (HttpStatusCode.OK, XmlOkResponse);
             }
             catch (MoParserException ex)
             {
@@ -63,7 +63,7 @@ public async Task ReceiveDeliveryReportAsync(HttpContext context)
             var body = await new StreamReader(context.Request.Body).ReadToEndAsync();
             var result = await ReceiveDeliveryReportAsync(body);
             context.Response.StatusCode = (int)result.status;
-            await new StreamWriter(context.Response.Body).WriteAsync(result.responseBody);
+            await HttpResponseWritingExtensions.WriteAsync(context.Response, result.responseBody);
         }
 
         public async Task<(HttpStatusCode status, string responseBody)> ReceiveDeliveryReportAsync(string requestBody)
@@ -78,7 +78,7 @@ public async Task ReceiveDeliveryReportAsync(HttpContext context)
             {
                 var drmessage = DrParser.Parse(document);
                 await drReceiver.Invoke(drmessage);
-                return (HttpStatusCode.OK, string.Empty);
+                return (HttpStatusCode.OK, XmlOkResponse);
             }
             catch (DrParserException ex)
             {

PSWin.Receiver/LinkMobility.PSWin.Receiver.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>netstandard2.0</TargetFramework>
     <GeneratePackageOnBuild>True</GeneratePackageOnBuild>
-    <Version>1.1.0</Version>
+    <Version>1.3.0</Version>
     <Company></Company>
     <Authors>Link Mobility NE</Authors>
     <Description>Receive mobile originated messages and delivery reports from PSWin gateway.</Description>