twisted.names.dns.Message should have a meaningful repr
|Reported by:||rwall||Owned by:||rwall|
branch-diff, diff-cov, branch-cov, buildbot
dns.Message is difficult to debug because it used the default repr.
In : from twisted.names.dns import Message In : m = Message() In : repr(m) Out: '<twisted.names.dns.Message instance at 0x2dca878>'
It would be nice to include all the fields values in the repr.
Easiest way is to inherit from FancyStrMixin
Which is what we've done with the proposed EDNSMessage class here:
Change History (14)
comment:5 Changed 3 years ago by hawkowl
- Keywords review removed
- Owner changed from hawkowl to rwall
comment:12 Changed 3 years ago by rwall
- Branch changed from branches/meaningful-message-repr-6847 to branches/meaningful-message-repr-6847-2
comment:13 in reply to: ↑ 11 Changed 3 years ago by rwall
- Owner changed from lewq to rwall
- Status changed from new to assigned