Ticket #5724 defect closed invalid

Opened 11 months ago

Last modified 11 months ago

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

Reported by: itamar Owned by:
Priority: normal Milestone:
Component: mail Keywords:
Cc: Branch:
Author: Launchpad Bug:

Description

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

Changed 11 months ago by exarkun

  • status changed from new to closed
  • resolution set to invalid

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.