Skip to content

Conversation

@moroine
Copy link
Contributor

@moroine moroine commented Sep 19, 2025

Updated the s3cmd installation command to include proxy environment variables.

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

Updated the s3cmd installation command to include proxy environment variables.
Copy link
Member

@BYK BYK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!!!

@BYK BYK enabled auto-merge (squash) September 19, 2025 13:47
@aminvakil
Copy link
Collaborator

This reminds me of a problem which I've faced a while ago, setting HTTP_PROXY, HTTPS_PROXY does not work, it should be lowercase.

There are lots of packages which work this way (either lowercase or uppercase), but maybe we can support both, this should be left for another PR though.

@BYK BYK merged commit 8fc3efa into getsentry:master Sep 19, 2025
11 checks passed
@aldy505
Copy link
Collaborator

aldy505 commented Sep 19, 2025

Oh wow I missed that. Thank you!!

@github-actions github-actions bot locked and limited conversation to collaborators Oct 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants