Opened 7 years ago

Closed 7 years ago

#5724 defect closed invalid (invalid)

twisted.protocols.smtp.SMTPClient has debug set to True by default, storing everything sent in memory

Reported by: Itamar Turner-Trauring Owned by:
Priority: normal Milestone:
Component: mail Keywords:
Cc: Branch:


Having SMTPClient.debug=True is a recipe for memory leaks. If we can't set it to False on base class for backwards compat reasons, we should at least disable it on the high-level APIs that don't expose that info anyway, I think.

Change History (1)

comment:1 Changed 7 years ago by Jean-Paul Calderone

Resolution: invalid
Status: newclosed

This only records the last 10 lines sent. A line in SMTP is required to be around 80 bytes maximum. SMTPClient has plenty of problems, but this isn't one afaict.

Note: See TracTickets for help on using tickets.