Fix VoiceWebSocket assuming 1000 is a fatal close code

Francessco121 · Oct 21, 2017 · a6ab4ca · a6ab4ca
1 parent 99abb66
commit a6ab4ca
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/Discore/Voice/Net/VoiceWebSocket.cs b/src/Discore/Voice/Net/VoiceWebSocket.cs
@@ -64,6 +64,9 @@ await base.DisconnectAsync(closeStatus, statusDescription, cancellationToken)
 
         protected override void OnCloseReceived(WebSocketCloseStatus closeStatus, string closeDescription)
         {
+            if (closeStatus == WebSocketCloseStatus.NormalClosure)
+                return;
+
             VoiceCloseCode voiceCloseCode = (VoiceCloseCode)closeStatus;
             switch (voiceCloseCode)
             {
@@ -78,7 +81,11 @@ protected override void OnCloseReceived(WebSocketCloseStatus closeStatus, string
                     OnNewSessionRequested?.Invoke(this, EventArgs.Empty);
                     break;
                 default:
-                    log.LogVerbose($"Fatal close code: {voiceCloseCode} ({(int)voiceCloseCode})");
+                    if ((int)voiceCloseCode >= 4000)
+                        log.LogVerbose($"Fatal close code: {voiceCloseCode} ({(int)voiceCloseCode}), {closeDescription}");
+                    else
+                        log.LogVerbose($"Fatal close code: {closeStatus} ({(int)closeStatus}), {closeDescription}");
+
                     OnUnexpectedClose?.Invoke(this, EventArgs.Empty);
                     break;
             }