async-email / async-smtp

Apache License 2.0
59 stars 12 forks source link

feat: Client connect to SOCKS5 stream #19

Closed amaury1093 closed 4 years ago

amaury1093 commented 4 years ago

This may be a long shot. In my project, I need to connect to a SMTP server via a SOCKS5 proxy. This PR adds this possibility.

I know that this might be adding maintenance for you, esp now with one new dependency (fast-socks5). If you close this PR, I totally understand, and I will just fork it and use my own fork.

Note: an idea to remove the fast-socks5 dep: Make NetworkStream a trait? It'll have Connector, fn shutdown(), fn peer_addr() etc.

amaury1093 commented 4 years ago

Friendly ping 🙏