Opened 2 years ago

Closed 2 years ago

#5783 regression closed fixed (fixed)

SSL support needs to be optional in twisted mail, again

Reported by: exarkun Owned by: exarkun
Priority: highest Milestone:
Component: mail Keywords:
Cc: Branch: branches/mail-without-ssl-5783
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

r34769 / #4963 introduced a hard dependency on OpenSSL in twisted.mail.tap, whereas previously it was an optional dependency. It should be optional again.

Change History (4)

comment:1 Changed 2 years ago by exarkun

  • Author set to exarkun
  • Branch set to branches/mail-without-ssl-5783

(In [34825]) Branching to 'mail-without-ssl-5783'

comment:2 Changed 2 years ago by exarkun

  • Keywords review added
  • Priority changed from normal to highest

comment:3 Changed 2 years ago by glyph

  • Keywords review removed
  • Owner set to exarkun

LGTM, as soon as the rest of the tests pass. Thanks for the review softball :-).

comment:4 Changed 2 years ago by exarkun

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

(In [34833]) Merge mail-without-ssl-5783

Author: exarkun
Reviewer: glyph
Fixes: #5783

Make twistd mail's SSL dependency optional again.

Note: See TracTickets for help on using tickets.