Opened 5 months ago
#6957 task new
Deprecate twisted.names.server.DNSServerFactory verbose logging
|Reported by:||rwall||Owned by:|
The verbose logging queries and responses by DNSServerFactory seems unnecessary and was probably introduced in lieu of unit tests when the class was first written.
The verbose query / response time measurement is more useful (especially for recursive and forwarding servers) but its implementation is ugly; tacking a timeReceived attribute onto the request message and response message and the logged message is unstructured, so the time data is difficult to analyse.
Lets deprecate the whole lot and introduce a more elegant way of recording query response time measurements later if anyone actually wants it.
This issue is raised as a result of #6645 where the request message is isolated from the response message instance.
See also #4021