#5425 defect new
Twisted provided ILogObserver implementations do not declare that they implement ILogObserver
|Reported by:||robertc||Owned by:|
Description (last modified by )
This means they cannot be passed through application.addComponent to customise the log observer used in a tac file; and they also provide a poor example for folk writing their own observers (e.g. the one I wrote for python-oops-twisted has an emit method which seemed idiomatic, but that emit method while it meets the contract for ILogObserver does not and can not declare that it implements it).
If the twisted internal code were idiomatic here (e.g. implements
__call__ and had an
implements(ILogObserver) declaration) we would have saved a bunch of developer time wondering wtf things were not working as expected.
Change History (3)
comment:2 Changed 5 years ago by
|Summary:||Twisted provided ILogObserver implementations do not implement ILogObserver → Twisted provided ILogObserver implementations do not declare that they implement ILogObserver|