Update MqttServer.cs (#1858)

* Update MqttServer.cs clientID is null => RetainedMessageChangedEventArgs ctor throw new ArgumentNullException(nameof(clientId) * Update ReleaseNotes.md --------- Co-authored-by: Christian <6939810+chkr1011@users.noreply.github.com>
dotnet · Nov 1, 2023 · e4f605f · e4f605f
1 parent 6fdee9e
commit e4f605f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/ReleaseNotes.md b/.github/workflows/ReleaseNotes.md
@@ -1,2 +1 @@
-* [Client] Fixed wrong TLS options handling (#1830).
-* [Client] Fixed NullReferenceExeption when performing a Ping when the client is not connected (#1831).
+* [Server] Fixed not working _UpdateRetainedMessageAsync_ public api (#1858, thanks to @kimdiego2098)
diff --git a/Source/MQTTnet/Server/MqttServer.cs b/Source/MQTTnet/Server/MqttServer.cs
@@ -362,7 +362,7 @@ public Task UpdateRetainedMessageAsync(MqttApplicationMessage retainedMessage)
             ThrowIfDisposed();
             ThrowIfNotStarted();
 
-            return _retainedMessagesManager?.UpdateMessage(null, retainedMessage);
+            return _retainedMessagesManager?.UpdateMessage(string.Empty, retainedMessage);
         }
 
         protected override void Dispose(bool disposing)
@@ -405,4 +405,4 @@ void ThrowIfStarted()
             }
         }
     }
-}
+}