Skip to content

Allow request headers and other arguments to pass through TDSCatalog #918

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

Closed
kahemker opened this issue Feb 28, 2020 · 0 comments · Fixed by #1766
Closed

Allow request headers and other arguments to pass through TDSCatalog #918

kahemker opened this issue Feb 28, 2020 · 0 comments · Fixed by #1766

Comments

@kahemker
Copy link
Contributor

kahemker commented Feb 28, 2020

Is your feature request related to a problem? Please describe.
If the user is working behind a corporate proxy that does not allow for secure SSL traffic to pass, forecast.get_processed_data does not work. You will see errors relating to SSL certificate verification failing.

Describe the solution you'd like
I created a fork for pvlib-python and siphon that show how to pass the necessary arguments all the way through into the session_manager within the TDSCatalog.

Describe alternatives you've considered
I don't see any alternatives. There does not seem to be the necessary global scope variables within urllib to set the necessary arguments. Unfortunately, until my pull request from Siphon is included into that release, my fork of pvlib-python will not run. I tested my new code by pulling in my forked branch from siphon and the test_forecast.py passes.

@CameronTStark CameronTStark added this to the 0.8.0 milestone Mar 2, 2020
@wholmgren wholmgren removed this from the 0.8.0 milestone Aug 28, 2020
@kandersolar kandersolar mentioned this issue Jun 12, 2023
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants