Make LMTP client return successful results as well

slimta / python-slimta

Python libraries to send, receive, and queue email.

https://slimta.org/

MIT License

171 stars 43 forks source link

Make LMTP client return successful results as well #131

Closed n11c closed 6 years ago

n11c commented 6 years ago

Not sure if there's a reason why the LMTP client was returning RelayErrors for failed deliveries but nothing for successful ones. Since the LMTP servers I work with actually send back a 250 reply on successful delivery, I'm assuming it is valid.

This small fix makes sure they're propagated back to the relay.

coveralls commented 6 years ago

Coverage decreased (-0.05%) to 92.983% when pulling c9910da8c8a4709eb7e71d5608b47e869286525e on oasiswork:lmtp_results into 2baab7dcdaa7f01b78ae1137651d077ec8a03f71 on slimta:master.

icgood commented 6 years ago

Good catch, thanks!