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

Backport of backends: Fix crash when interrupting during interactive prompt for values' into v1.11 #36460

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Feb 7, 2025

Backport

This PR is auto-generated from #36448 to be assessed for backporting due to the inclusion of the label 1.11-backport.

The below text is copied from the body of the original PR.


This PR fixes a crash that affects all remote state backends that implement the backendbase.Base. When interrupting (Ctrl+c) during Terraform's interactive value prompt, we convert the resulting unknown value to a null value.
Our generic PrepareConfig wasn't prepared to handle nulls and crashed when applying the sdk-like defaults.

Fixes #36420

Target Release

1.11.x

CHANGELOG entry

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

Overview of commits

@github-actions github-actions bot force-pushed the backport/dbanck/init-sigint-crash/factually-lasting-gopher branch from c8e2edb to 37eb6eb Compare February 7, 2025 09:41
@github-actions github-actions bot requested a review from dbanck February 7, 2025 09:41
@dbanck dbanck marked this pull request as ready for review February 7, 2025 09:46
@dbanck dbanck requested a review from a team as a code owner February 7, 2025 09:46
@dbanck dbanck merged commit aafd40d into v1.11 Feb 7, 2025
7 checks passed
@dbanck dbanck deleted the backport/dbanck/init-sigint-crash/factually-lasting-gopher branch February 7, 2025 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant