Skip to content

fix: remove hardcoded secret in main (CWE-798) - #29

Closed
anupamme wants to merge 1 commit into
pre-commit-ci:mainfrom
anupamme:fix-repo-lite-action-cwe-798-runtime-token-cli-exposure
Closed

anupamme wants to merge 1 commit into
pre-commit-ci:mainfrom
anupamme:fix-repo-lite-action-cwe-798-runtime-token-cli-exposure

Conversation

@anupamme

Copy link
Copy Markdown

The CLI application accepts a --runtime-token argument via command line, making the sensitive token visible in process listings (ps aux), shell history files, and CI/CD logs. This exposure allows unauthorized parties with local system access or log access to steal the token and impersonate the CI runner. The affected code is bin/main:236. This change is the fix I would apply.

Reference: CWE-798

What changed

  • bin/main
  • main.mjs

Verification

No automated check could be run against this repository, so this change is unverified beyond review. Please treat it as a suggestion.


Automated security fix by OrbisAI Security

Automated security fix generated by OrbisAI Security
@asottile asottile closed this Sep 29, 2026
@pre-commit-ci pre-commit-ci locked as spam and limited conversation to collaborators Sep 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants