Remove logging since this code path is essentially untestable.

Reverts part of the original fix from #60 (we can't get this path covered with reasonable effort).
OmniSharp · Jan 11, 2018 · 86226b7 · 86226b7
1 parent 39167aa
commit 86226b7
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/Server/LspRequestRouter.cs b/src/Server/LspRequestRouter.cs
@@ -123,11 +123,9 @@ public async Task<ErrorResponse> RouteRequest(IHandlerDescriptor descriptor, Req
                     {
                         @params = request.Params?.ToObject(descriptor.Params, _serializer.JsonSerializer);
                     }
-                    catch (Exception cannotDeserializeRequestParams)
+                    catch
                     {
-                        _logger.LogError(new EventId(-32602), cannotDeserializeRequestParams, "Failed to deserialise request parameters.");
-
-                        return new InvalidParams(request.Id);
+                            return new InvalidParams(request.Id);
                     }
 
                     result = ReflectionRequestHandlers.HandleRequest(descriptor, @params, cts.Token);