Skip to content
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

Add option to authenticate via proxies and with verification cerificates #761

Open
LucaMingarelli opened this issue Aug 3, 2023 · 2 comments

Comments

@LucaMingarelli
Copy link

It is currently not possible to pass proxies and verify parameters upon connection. This imposes substantial restrictions to data access for users working in institutions under proxies.

It would be an easy change amounting to including default parameters proxies=None and verify=True to a number of functions such as

  • All calls to requests
  • pyam.iiasa.Connection so that this can be called e.g. as conn = pyam.iiasa.Connection(proxies=PROXIES['https'], verify=where())
  • pyam.read_iiasa
  • etc

Would you kindly be able to push a new version accounting for this?

@znicholls
Copy link
Collaborator

I believe PRs are welcome so feel free to contribute the changes needed and ask for a review

@danielhuppmann
Copy link
Member

Thanks @LucaMingarelli - we don't have the capacity to implement and thoroughly test such a feature extension ourselves, but happy to review if you can start a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants