Skip to content

Added a User-Agent header for remote-state http backend client #34662

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 1 commit into
base: main
Choose a base branch
from

Conversation

NetMoose
Copy link

Added a User-Agent header for the http client to determine which client call to the http backend for change state.

Sometimes need to be able to send specially formatted state in different ways (for example, hiding sensitive data that will be shown in Swagger output but not hidden for terraform cli). In this case, would like to be able to identify which client sends the request for change state.

Target Release

1.8.x

Draft CHANGELOG entry

ENHANCEMENTS

  • When calling http-backend from terraform, "User-Agent" header does not change, which does not allow correctly identifying the type of connecting client. Changed "User-Agent" header from "Go-http-client/1.1" to "Terraform-http-client/1.1"

Added a User-Agent header for the http client to determine which client call to the backend.
@NetMoose NetMoose requested a review from a team as a code owner February 13, 2024 04:51
@hashicorp-cla
Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@crw
Copy link
Contributor

crw commented Feb 13, 2024

Thanks for this submission. Please sign the CLA per #34662 (comment) and I will bring this to the triage meeting. Thanks!

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.

3 participants