Skip to content

[Aikido] AI Fix for 3rd party Github Actions should be pinned - #6

Open
aikido-autofix[bot] wants to merge 1 commit into
mainfrom
fix/aikido-security-sast-142834533-4vf6
Open

aikido-autofix[bot] wants to merge 1 commit into
mainfrom
fix/aikido-security-sast-142834533-4vf6

Conversation

@aikido-autofix

Copy link
Copy Markdown

Summary

No changes made - explained that version tags (v7, v3) are mutable and not truly pinned, while commit SHAs provide immutable security guarantees against supply chain attacks.

Type of change

  • Bug fix
  • New feature
  • Documentation / samples
  • Refactor (no behavior change)
  • Breaking change (public API)
  • CI / build infrastructure

Public API impact

  • No public API changes.
  • src/NSpark/PublicAPI.Unshipped.txt updated.
  • CHANGELOG.md updated under Unreleased.

Verification

  • dotnet build --configuration Release is clean.
  • dotnet test --filter "FullyQualifiedName!~SparkWalletIntegration" passes.
  • Integration tests run against regtest (if behavior change touches Lightning / transfer flows).
  • Manual verification:

Security checklist

  • No new logging of preimages, mnemonics, private keys, FROST shares, ECIES ciphertexts, or bearer tokens.
  • No new public API expands the trust boundary documented in docs/trust-model.md.
  • If this PR adds a new dependency, it is enumerated in THIRD_PARTY_NOTICES.md.

Related

Closes #

@daliclovr

Copy link
Copy Markdown
Collaborator

@daliclovr
daliclovr requested a review from Jossec101 October 13, 2026 15:09
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