-
Notifications
You must be signed in to change notification settings - Fork 187
Update default value for Terraform MCP Server integration #2113
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
Conversation
Changed default value for terraform.mcp.server.enable from true to false.
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ from team-prodsec, we should disable this option by default.
Thanks @becojo for the PR.
edit after triggering the pipeline: I believe the only thing that is missing to make the CI checks happy is a changelog entry in .changes/unreleased
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Could you please add a new changelog entry for this? You can generate it by running changie new
and following the prompts. Thanks!
Great decision - I didn't appreciate an MCP server image magically appearing in my local |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
#2101 #2103
Changed default value for
terraform.mcp.server.enable
from true to false.PCI review checklist
Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.