Method OnExceptionThrownAsync() in DnsServer.cs should not call Trace.TraceError()...

Feb 10, 2016 at 3:15 PM
Imho the call to Trace.TraceError() inside DnsServer.OnExceptionThrownAsync() should be deleted.
The user can handle logging and tracing using the event ExceptionThrown.

Sockets can be closed by a calling client (-> SocketException: ConnectionReset), typically if the custom DnsServer is the last in a row of DnsServers called asynchronously by a client, this client could receive a positive answer from the first DNS after it made the connection to the custom DNS and therefor just close all pending request, i.e. close the connection. But this would be a "standard" behaviour, which one normally don't want to include in a trace.