[offlineimap] Proxy support (#169)

Nicolas Sebrecht notifications at github.com
Fri Feb 13 12:49:17 GMT 2015


First, I've worked with a China company in the past and I know that having to workaround the filters can seriously become boring, even more for Chinese. I join your efforts to bypass them. You might add the comment directly in the source code.

* The implementation note should not stand in the commit message and unless I've missed something both branches look unrelated and independent.
* Commits must be signed-off-by.
* Both patches should be squashed.
* The option is not described in `offlineimap.conf` and should express the dependency requirement.
* If the library import of socks fails, we should provide a well-written message rather than let the exception bubble up.

Other than that, the code looks correct to me while I did not test this feature.

---
Reply to this email directly or view it on GitHub:
https://github.com/OfflineIMAP/offlineimap/pull/169#issuecomment-74248719
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/offlineimap-project/attachments/20150213/b416a42e/attachment-0003.html>


More information about the OfflineIMAP-project mailing list