Remove empty catch block. The issue with Flush method was fixed in

the version 2.6.0-Beta4 of ApplicationInsights.
See microsoft/ApplicationInsights-dotnet#755

Author: Valerii Sarkisov

src/Serilog.Sinks.ApplicationInsights/Sinks/ApplicationInsights/ApplicationInsightsSinkBase.cs

@@ -199,13 +199,8 @@ protected virtual void Dispose(bool disposeManagedResources)
                     // attempt to free managed resources
                     try
                     {
-                        // this apparently (see https://github.com/serilog/serilog-sinks-applicationinsights/issues/46#issuecomment-379218037) throws a NullReference Exception on app shutdown..
                         _telemetryClient?.Flush();
                     }
-                    catch (Exception)
-                    {
-                        // .. and as ugly as THIS is, .Dispose() methods shall not throw exceptions
-                    }
                     finally
                     {
                         _telemetryClient = null;