Ticket #2158 enhancement closed fixed

Opened 7 years ago

Last modified 7 years ago

SMTP server incorrectly handles SMTPBadSender from validateFrom

Reported by: exarkun Owned by:
Priority: highest Milestone:
Component: mail Keywords:
Cc: Branch:
Author: Launchpad Bug:

Description

It first tells the client that the address was rejected. It next tells the client that the address was accepted.

Change History

1

Changed 7 years ago by exarkun

  • keywords review added
  • priority changed from normal to highest

Fixed in validatefrom-2158

2

Changed 7 years ago by exarkun

  • owner exarkun deleted

3

Changed 7 years ago by therve

  • keywords review removed
  • owner set to exarkun

Looks good. Just one typo in _ebAuthenticated docstring ('tra nslating').

4

Changed 7 years ago by exarkun

Fixed that docstring. Harsh light of day motivates me to make a couple further exception handling cleanups in this branch, though.

5

Changed 7 years ago by exarkun

  • owner exarkun deleted
  • keywords review added

Okay, cleaned up a bit and added some more tests. This also fixes response decode error handling now.

6

Changed 7 years ago by washort

  • keywords review removed
  • owner set to exarkun

Looks good, tests pass, and it fixes the problem. Code looks simpler, which is nice.

7

Changed 7 years ago by exarkun

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

(In [18417]) Merge validatefrom-2158

Author: exarkun Reviewer: therve, washort Fixes #2158

Add tests for handling of various IMessageDelivery implementation behaviors, including exceptions from validateFrom. Also add tests for failures from portal.login and base64 decoding of challenge response strings. Fix bugs revealed by all of these tests.

8

Changed 2 years ago by <automation>

  • owner exarkun deleted
Note: See TracTickets for help on using tickets.