Opened 10 months ago

#6957 task new

Deprecate twisted.names.server.DNSServerFactory verbose logging

Reported by: rwall Owned by:
Priority: normal Milestone:
Component: names Keywords:
Cc: Branch:
Author: Launchpad Bug:

Description

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

Change History (0)

Note: See TracTickets for help on using tickets.