Skip to content

fix(terraform-static-analysis): workaround TFLint issue #2591 attestation panic - #253

Merged
Aaron Robinson (ASTRobinson) merged 2 commits into
mainfrom
fix/tflint-2591-workaround
Jul 17, 2026
Merged

fix(terraform-static-analysis): workaround TFLint issue #2591 attestation panic#253
Aaron Robinson (ASTRobinson) merged 2 commits into
mainfrom
fix/tflint-2591-workaround

Conversation

@ASTRobinson

@ASTRobinson Aaron Robinson (ASTRobinson) commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR applies a temporary workaround for terraform-linters/tflint#2591, where tflint --init panics in the attestation verification path due to GitHub's API now returning bundle: null and only a bundle_url in attestation list responses.

Changes

Attestation panic workaround (#2591)

  • Run TFLint commands with GITHUB_TOKEN unset in the action runtime to avoid the crashing attestation path. GITHUB_TOKEN is still available later for PR comments.
  • Set signature = "pgp" in bundled TFLint plugin configs so verification uses the PGP path instead of the broken attestation path.
  • Fail fast with a clear error message if tflint --init exits non-zero.

Plugin version bumps

Plugin Old New
tflint-ruleset-aws 0.33.0 0.48.0
tflint-ruleset-azurerm 0.27.0 0.32.0
tflint-ruleset-terraform 0.9.1 0.15.0

Configs updated: tflint.aws.hcl, tflint.azure.hcl, tflint.default.hcl

Docs

  • terraform-static-analysis/README.md — noted the temporary signature = "pgp" workaround and linked to the upstream fix PR.

Validation

  • bash -n terraform-static-analysis/entrypoint.sh — no syntax errors.
  • Editor diagnostics on all touched files — no errors.

Follow-up

Remove the signature = "pgp" lines and the env -u GITHUB_TOKEN wrapper once the upstream fix ships:

@ASTRobinson
Aaron Robinson (ASTRobinson) requested a review from a team as a code owner July 17, 2026 09:17

@mikereiddigital mikereiddigital left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lg2m

@mikereiddigital mikereiddigital left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lg2m

@ASTRobinson
Aaron Robinson (ASTRobinson) merged commit 0d6fd59 into main Jul 17, 2026
4 checks passed
@ASTRobinson
Aaron Robinson (ASTRobinson) deleted the fix/tflint-2591-workaround branch July 17, 2026 11:07
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.

2 participants