Skip to content

Fix #36897: Resolve NO_PROXY recognition issue in oss remote backend #36898

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

xingjiu
Copy link

@xingjiu xingjiu commented Apr 17, 2025

Fixes #36897

This PR addresses [Terraform Issue #36897]. The core implementation was previously contributed to OpenTofu PR #2675 and is adapted here to fit Terraform's codebase.

Target Release

1.13.x

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@xingjiu xingjiu requested a review from a team as a code owner April 17, 2025 07:12
Copy link

hashicorp-cla-app bot commented Apr 17, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

Changelog Warning

Currently this PR would target a v1.13 release. Please add a changelog entry for in the .changes/v1.13 folder, or discuss which release you'd like to target with your reviewer. If you believe this change does not need a changelog entry, please add the 'no-changelog-needed' label.

@jbardin
Copy link
Member

jbardin commented Apr 17, 2025

@xiaozhu36 per codeowners

@xiehan
Copy link
Member

xiehan commented Jun 24, 2025

@xiaozhu36 Friendly reminder to please review this backend PR. If we want to include this in the 1.13 release, it needs to be approved and merged within the next couple of weeks.

@xiaozhu36
Copy link
Contributor

@xiehan I have reviewed the PR and there is no problem.

@xiehan
Copy link
Member

xiehan commented Jun 26, 2025

Great! @xingjiu Could you rebase your PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OSS backend does not recognize NO_PROXY environment variable, causing unintended proxy usage
5 participants