Skip to content

Environment support #33

Closed
Closed

Description

@lovelydinosaur

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.

Activity

  1. changed the title [-]Environment support.[/-] [+]Environment support[/+] on May 2, 2019
  2. jhtimmins commented on Jul 24, 2019

    @jhtimmins

    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.

  3. sethmlarson commented on Jul 24, 2019

    @sethmlarson
    Contributor

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

  4. cansarigol commented on Jul 30, 2019

    @cansarigol
    Contributor

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

  5. sethmlarson commented on Aug 17, 2019

    @sethmlarson
    Contributor

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

  6. florimondmanca commented on Aug 26, 2019

    @florimondmanca
    Contributor

    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?

  7. sethmlarson commented on Aug 27, 2019

    @sethmlarson
    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.

  8. florimondmanca commented on Aug 30, 2019

    @florimondmanca
    Contributor

    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.

  9. jlaine commented on Aug 30, 2019

    @jlaine

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

  10. florimondmanca commented on Aug 30, 2019

    @florimondmanca
    Contributor

    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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions