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

Expose existing boto3 proxy configuration options #9045

Open
mogul opened this issue Jan 19, 2022 · 6 comments
Open

Expose existing boto3 proxy configuration options #9045

mogul opened this issue Jan 19, 2022 · 6 comments
Labels
cross-sdk feature-request A feature should be added or improved. p3 This is a minor priority issue

Comments

@mogul
Copy link

mogul commented Jan 19, 2022

Is your feature request related to a problem? Please describe.

I have configured the AWS CLI to use a proxy by setting the HTTPS_PROXY environment variable. However the proxy presents an internal certificate and I see a message saying "Cannot connect to proxy". (I have verified that the proxy is usable with curl by testing that I can connect and that the certificate presented is valid.)

There is no way to configure boto3's existing proxy_ca_bundle option, whether by environment variable or command flag.

Describe the solution you'd like

  • I want the aws command to make use of anAWS_S3_PROXY_CA_BUNDLE environment variable when it is set
  • I want the s3 subcommand to have an explicit --proxy_ca_bundle flag

Describe alternatives you've considered
I've tried setting the value in a dictionary in the config file, but it's not working and there's no example to refer to, so I'm not sure if there's a problem with the way I'm specifying the value, or with the value itself, or if this option is not expected to work.

Additional context
The proxy restricts egress traffic to expected S3 buckets in order to prevent compromised applications from exfiltrating data or establishing a command-and-control backchannel (eg SolarWinds and log4shell-style attacks).

I also checked the v2 branch to see if the option was there, but didn't find any indication that this setting is on anyone's radar.

@mogul mogul added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Jan 19, 2022
@mogul
Copy link
Author

mogul commented Jan 19, 2022

I submitted a PR on botocore that would implement the environment variable option.

@tim-finnigan
Copy link
Contributor

Hi @mogul, thanks for the feature request and PR. I brought this up for discussion with the team and it is something we are looking into further. We will let you know when we have an update.

@tim-finnigan tim-finnigan added needs-review This issue or pull request needs review from a core team member. and removed needs-triage This issue or PR still needs to be triaged. labels Jan 21, 2022
@mogul
Copy link
Author

mogul commented Jan 21, 2022 via email

@mogul
Copy link
Author

mogul commented Apr 26, 2022

Hi @tim-finnigan... Any updates on this one?

@tim-finnigan
Copy link
Contributor

Thanks @mogul for the feature request, I left a comment on your PR. We can keep this issue open for tracking the request, and others can add a 👍 to indicate their support or leave a comment to provide additional feedback.

@tim-finnigan tim-finnigan removed the needs-review This issue or pull request needs review from a core team member. label May 3, 2022
@tim-finnigan
Copy link
Contributor

As mentioned previously on the PR this type of work would need to be coordinated across SDKs, so the broader teams would need to review and decide on this. I found a related backlog item on this topic: aws/aws-sdk#127, but I'm not sure if there's enough overlap in your requests to consider this a duplicate. But I'm going to transfer this issue to that repository as well since that is where cross-SDK feature requests are tracked. And we continue to encourage others to leave feedback on their use cases related to this or 👍 the issue. Thanks!

@tim-finnigan tim-finnigan transferred this issue from aws/aws-cli Sep 28, 2022
@tim-finnigan tim-finnigan self-assigned this Sep 28, 2022
@tim-finnigan tim-finnigan removed their assignment Sep 28, 2023
@RanVaknin RanVaknin assigned jmklix and unassigned khushail Mar 1, 2024
@jmklix jmklix added the p3 This is a minor priority issue label Aug 29, 2024
@jmklix jmklix transferred this issue from aws/aws-sdk Oct 30, 2024
@jmklix jmklix removed their assignment Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-sdk feature-request A feature should be added or improved. p3 This is a minor priority issue
Projects
None yet
Development

No branches or pull requests

4 participants