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

Environment support #33

Closed
tomchristie opened this issue May 2, 2019 · 9 comments · Fixed by #353
Closed

Environment support #33

tomchristie opened this issue May 2, 2019 · 9 comments · Fixed by #353
Labels
good first issue Good for newcomers

Comments

@tomchristie
Copy link
Member

Needs to handle anything that requests currently deals with when trust_env=True is set.
.netrc, REQUEST_CA_BUNDLE and anything else relevant.

First step to dealing with this would be to outline exactly what set of stuff requests includes. If anyone wants to dig into this and comment on this issue comprehensively with what set of behavior we need, then that'd be a great start.

We have adapters/environment.py currently stubbed out to deal with adding in this behavior.

@tomchristie tomchristie changed the title Environment support. Environment support May 2, 2019
@sethmlarson sethmlarson added the good first issue Good for newcomers label Jul 18, 2019
@jhtimmins
Copy link

If this is still important, I'd be interested in doing some research here to determine what exactly requests includes, and then possibly implementing proper functionality.

@sethmlarson
Copy link
Contributor

This is definitely important! If you can lend a hand with discovery that'd be great.

@cansarigol
Copy link
Contributor

Hi, I tried to include netrc support. #177 what do you think about this approach?

@sethmlarson
Copy link
Contributor

We still need a way to grab environment variables and to document those environment variables for this issue to be closed.

@florimondmanca
Copy link
Member

florimondmanca commented Aug 26, 2019

I did some research in the Requests docs looking for environment variables it supports, and whether we support them already:

(Note that the names of the *_PROXY environment variables seems be a convention.)

To make extra sure we don't miss anything, it would probably be worth cloning the Requests repo and grepping our way using environment variable, os.getenv and os.environ.get.

The scope of this issue is rather broad, so if that's confirmed and as suggested by @sethmlarson in #277 (comment), how about creating issues for each set of environment variables?

@sethmlarson
Copy link
Contributor

Should also support SSL_CERT_FILE as that's merged in Requests 3.0 branch and was intended to be supported and is an OpenSSL thing. See PEP 476.

@florimondmanca
Copy link
Member

After inspecting the Requests code base, I updated the list with CURL_CA_BUNDLE — I couldn't find any other use of environment variables in there.

@jlaine
Copy link

jlaine commented Aug 30, 2019

Having recently spent a lot of time in wireshark, I'd quite like SSLKEYLOGFILE too (unless Python's ssl provides this free of charge?)

@florimondmanca
Copy link
Member

Having recently spent a lot of time in wireshark, I'd quite like SSLKEYLOGFILE too (unless Python's ssl provides this free of charge?)

Seems it will be landing in Python 3.8. Meanwhile, sslkeylog?

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

Successfully merging a pull request may close this issue.

6 participants