-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does not work behind a proxy #6
Comments
Will rewrite api to use requests :) |
I just forked the project and try to contribute some code. |
Awesome, I'll leave it in you then. The change should be simple. I wasn't aware of requests when i wrote the plugin, that's why it is build with urllib :) |
the thing i'm not sure about is that requests is not a python standard lib. this will blow up the plugin and i'm not quite sure how to ship 3rd party libraries within the plugin. |
http://stackoverflow.com/a/15180938 |
Any progress with including requests bundle into the plugin? :) |
i did not event have time to follow your link :-) if you can do it earlier, i'd really appreciate |
Because
urllib
does not use system proxy by default, this will not work in a proxied environment. If you start from command line and use thecurl
fallback, it works fine.requests
instead ofurllib
which automatically reuses proxy environmentwhat do you think?
The text was updated successfully, but these errors were encountered: