Change unhandled messages to warnings instead of errors (#767)

rjmholt · web-flow · commit f0cb9fa7db1e · 2018-10-11T09:29:48.000-07:00
diff --git a/src/PowerShellEditorServices.Protocol/MessageProtocol/MessageDispatcher.cs b/src/PowerShellEditorServices.Protocol/MessageProtocol/MessageDispatcher.cs
@@ -131,7 +131,7 @@ public async Task DispatchMessage(
                 else
                 {
                     // TODO: Message not supported error
-                    this.logger.Write(LogLevel.Error, $"MessageDispatcher: No handler registered for Request type '{messageToDispatch.Method}'");
+                    this.logger.Write(LogLevel.Warning, $"MessageDispatcher: No handler registered for Request type '{messageToDispatch.Method}'");
                 }
             }
             else if (messageToDispatch.MessageType == MessageType.Event)
@@ -144,13 +144,13 @@ public async Task DispatchMessage(
                 else
                 {
                     // TODO: Message not supported error
-                    this.logger.Write(LogLevel.Error, $"MessageDispatcher: No handler registered for Event type '{messageToDispatch.Method}'");
+                    this.logger.Write(LogLevel.Warning, $"MessageDispatcher: No handler registered for Event type '{messageToDispatch.Method}'");
                 }
             }
             else
             {
                 // TODO: Return message not supported
-                this.logger.Write(LogLevel.Error, $"MessageDispatcher received unknown message type of method '{messageToDispatch.Method}'");
+                this.logger.Write(LogLevel.Warning, $"MessageDispatcher received unknown message type of method '{messageToDispatch.Method}'");
             }
 
             if (handlerToAwait != null)

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ public async Task DispatchMessage(`
`131`	`131`	`else`
`132`	`132`	`{`
`133`	`133`	`// TODO: Message not supported error`
`134`		`- this.logger.Write(LogLevel.Error, $"MessageDispatcher: No handler registered for Request type '{messageToDispatch.Method}'");`
	`134`	`+ this.logger.Write(LogLevel.Warning, $"MessageDispatcher: No handler registered for Request type '{messageToDispatch.Method}'");`
`135`	`135`	`}`
`136`	`136`	`}`
`137`	`137`	`else if (messageToDispatch.MessageType == MessageType.Event)`
`@@ -144,13 +144,13 @@ public async Task DispatchMessage(`
`144`	`144`	`else`
`145`	`145`	`{`
`146`	`146`	`// TODO: Message not supported error`
`147`		`- this.logger.Write(LogLevel.Error, $"MessageDispatcher: No handler registered for Event type '{messageToDispatch.Method}'");`
	`147`	`+ this.logger.Write(LogLevel.Warning, $"MessageDispatcher: No handler registered for Event type '{messageToDispatch.Method}'");`
`148`	`148`	`}`
`149`	`149`	`}`
`150`	`150`	`else`
`151`	`151`	`{`
`152`	`152`	`// TODO: Return message not supported`
`153`		`- this.logger.Write(LogLevel.Error, $"MessageDispatcher received unknown message type of method '{messageToDispatch.Method}'");`
	`153`	`+ this.logger.Write(LogLevel.Warning, $"MessageDispatcher received unknown message type of method '{messageToDispatch.Method}'");`
`154`	`154`	`}`
`155`	`155`
`156`	`156`	`if (handlerToAwait != null)`