Skip to content

Conversation

@danenania
Copy link
Contributor

Testing internal PR scanning flow with staging action

- name: Run Promptfoo Code Scan
uses: promptfoo/code-scan-action@v0
uses: promptfoo/code-scan-action@staging

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow Medium

Unpinned 3rd party Action 'Promptfoo Code Scan' step
Uses Step
uses 'promptfoo/code-scan-action' with ref 'staging', not a pinned commit hash
app.get('/search', (req, res) => {
const term = req.query.q;
// XSS vulnerability - rendering unsanitized input
res.send(`<h1>Results for: ${term}</h1>`);

Check failure

Code scanning / CodeQL

Reflected cross-site scripting High

Cross-site scripting vulnerability due to a
user-provided value
.
Copy link

@promptfoo-scanner-staging promptfoo-scanner-staging bot left a comment

Choose a reason for hiding this comment

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

👍 All Clear

I reviewed this PR for LLM security vulnerabilities across the six defined classes (Prompt Injection, Data Exfiltration, PII/Secrets in Prompts, Insecure Output Handling, Excessive Agency, and Jailbreak Risks). The changes include a GitHub Actions workflow version update and a test file with an Express.js route handler. No LLM-related code or data flows were identified in this PR.

Minimum severity threshold for this scan: 🟡 Medium | Learn more


Was this helpful?  👍 Yes  |  👎 No 

@danenania
Copy link
Contributor Author

@promptfoo-scanner-staging

1 similar comment
@danenania
Copy link
Contributor Author

@promptfoo-scanner-staging

Copy link

@promptfoo-scanner-staging promptfoo-scanner-staging bot left a comment

Choose a reason for hiding this comment

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

👍 All Clear

I reviewed this PR for LLM-specific security vulnerabilities. The changes include a GitHub Actions workflow version update and a test file with a traditional XSS vulnerability. No LLM security issues were identified.

Minimum severity threshold for this scan: 🟡 Medium | Learn more


Was this helpful?  👍 Yes  |  👎 No 

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