Skip to content

Commit 1b18ac9

Browse files
authored
Resolve ConvertPayloadToString resulting in ArgumentNullException (#1745)
Resolve breaking change where previously ConvertPayloadToString returned null if the payload was null
1 parent 2cdad66 commit 1b18ac9

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

Source/MQTTnet/MqttApplicationMessageExtensions.cs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
using System;
66
using System.Text;
7+
using MQTTnet.Internal;
78

89
namespace MQTTnet
910
{
@@ -16,6 +17,11 @@ public static string ConvertPayloadToString(this MqttApplicationMessage applicat
1617
throw new ArgumentNullException(nameof(applicationMessage));
1718
}
1819

20+
if(applicationMessage.PayloadSegment == EmptyBuffer.ArraySegment)
21+
{
22+
return null;
23+
}
24+
1925
var payloadSegment = applicationMessage.PayloadSegment;
2026
return Encoding.UTF8.GetString(payloadSegment.Array, payloadSegment.Offset, payloadSegment.Count);
2127
}

0 commit comments

Comments
 (0)