Skip to content

Comments

Add SEMGREP_APPTOKEN to secrets.vuln#7

Open
kyle-semgrep wants to merge 1 commit intomainfrom
new-secrets-test
Open

Add SEMGREP_APPTOKEN to secrets.vuln#7
kyle-semgrep wants to merge 1 commit intomainfrom
new-secrets-test

Conversation

@kyle-semgrep
Copy link
Owner

@kyle-semgrep kyle-semgrep commented Jan 22, 2026

Note

Adds a new secrets.vuln file containing a plaintext SEMGREP_APPTOKEN value.

Written by Cursor Bugbot for commit 72dc042. This will update automatically on new commits. Configure here.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 72dc042649

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@@ -0,0 +1 @@
SEMGREP_APPTOKEN=4170b6b961c04bb13814358410366231f618349f45b360209ea70b962fb620ea

Choose a reason for hiding this comment

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

P1 Badge Remove committed SEMGREP_APPTOKEN secret

The new file commits a real-looking SEMGREP_APPTOKEN value directly into the repository, which exposes a credential to anyone with repo access and to any downstream forks or artifact caches. This is a security risk in any environment where the token is valid; the token should be removed from version control and loaded via a secret manager or environment variable instead.

Useful? React with 👍 / 👎.

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