Opened 2 years ago

Closed 2 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 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)

comment:1 Changed 2 years ago by exarkun

  • Resolution set to invalid
  • Status changed from new to closed

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.