Skip to content

[oblt-aw][security] Fix SEC-032 actionlint installer integrity verification#994

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
copilot/fix-sec-032-issue-992-4d6e1d1aa0640e87
Draft

[oblt-aw][security] Fix SEC-032 actionlint installer integrity verification#994
github-actions[bot] wants to merge 1 commit into
mainfrom
copilot/fix-sec-032-issue-992-4d6e1d1aa0640e87

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Closes #992

This change remediates a SEC-032 finding in the detector bootstrap installer flow by removing direct execution of network-fetched script content and enforcing integrity verification before execution.

What changed

  • Updated scripts/obs/install_security_detector_tools.sh:
    • kept pinned installer source commit (ACTIONLINT_DOWNLOAD_SCRIPT_SHA)
    • added ACTIONLINT_DOWNLOAD_SCRIPT_SHA256
    • replaced bash <(curl ...) with explicit curl -o download to download-actionlint.bash
    • added sha256sum -c - verification gate
    • execute installer only after checksum passes
    • remove temporary installer script after execution

Plan checklist

  • Read and execute SEC-032 remediation steps in order
  • Remove remote process-substitution execution
  • Add cryptographic integrity verification before execution
  • Re-run validations

Validation evidence

  • npm test passes (6/6)
  • SEC-032 detector heuristic check for shell scripts (curl|wget without integrity tokens) reports no remaining findings

Security requirements confirmation

  • Least privilege: No workflow/job permissions were expanded; this fix is confined to a shell script installer path.
  • Env indirection: No secrets/tokens are interpolated into command strings in this change; commands operate on non-secret constants and local files only.

Note

🔒 Integrity filter blocked 12 items

The following items were blocked because they don't meet the GitHub integrity level.

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

What is this? | From workflow: Observability Agentic Workflow Entrypoint

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

Replace process-substitution execution of remote installer with explicit download plus SHA-256 verification before execution.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

[oblt-aw][security] SEC-032 — findings (2026-05-21)

0 participants