Review comments

cvium · Mar 6, 2019 · 394d23a · 394d23a
1 parent 446f9bf
commit 394d23a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Emby.Server.Implementations/WebSockets/WebSocketManager.cs b/Emby.Server.Implementations/WebSockets/WebSocketManager.cs
@@ -65,7 +65,7 @@ private async Task ProcessMessage(byte[] messageBytes, TaskCompletionSource<bool
                 ? Encoding.UTF8.GetString(messageBytes, 0, messageBytes.Length)
                 : Encoding.ASCII.GetString(messageBytes, 0, messageBytes.Length);
 
-            // All messages are expected to be json
+            // All messages are expected to be valid JSON objects
             if (!message.StartsWith("{", StringComparison.OrdinalIgnoreCase))
             {
                 _logger.LogDebug("Received web socket message that is not a json structure: {Message}", message);

diff --git a/MediaBrowser.Api/Playback/Progressive/BaseProgressiveStreamingService.cs b/MediaBrowser.Api/Playback/Progressive/BaseProgressiveStreamingService.cs
@@ -362,6 +362,7 @@ private async Task<object> GetStreamResult(StreamRequest request, StreamState st
 
             var contentType = state.GetMimeType(outputPath);
 
+            // TODO: The isHeadRequest is only here because ServiceStack will add Content-Length=0 to the response
             // Headers only
             if (isHeadRequest)
             {