Updated MailHandler to allow using TLS without having a keyfile/certfile #47

Closed
wants to merge 10 commits into from

2 participants

@shanx

Hey guys,

Thanks for logbook! After dealing with the pain of python's stdlib logging for years, this library is such a relief! Never want to go back ;)

Attached is a small patch to allow using TLS without explicitly defining keyfile/certfile. I didn't find any unit tests testing this behavior. Also fixed a small typo.

Cheers,
Remco

@shanx

Hmm this is of course not really backwards compatible with people using tls with smtp in the old way. Let me know what you think and I could fix backward compatibility.

@brainstorm

Hello @shanx, I find your patch interesting, could you update it a little bit so that it merges the current codebase and guarantee backwards compatibility?

Thanks!

@brainstorm

Unmantained, closing.

@brainstorm brainstorm closed this Apr 18, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment