Skip to content

Conversation

@danenania
Copy link
Contributor

Testing fork PR scanning with SQL injection vulnerability

@promptfoo-scanner-staging
Copy link

Code Scan

Fork PR scanning is disabled by default. To scan this PR:

  • A maintainer can request a review from @promptfoo-scanner
  • Or enable automatic fork scanning in your workflow:
- uses: promptfoo/code-scan-action@v1
  with:
    enable-fork-prs: true

@danenania
Copy link
Contributor Author

@promptfoo-scanner-staging

@promptfoo-scanner-staging
Copy link

❌ Failed to trigger code scan. The workflow file promptfoo-code-scan.yml may not exist or may not have workflow_dispatch enabled.

@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 security vulnerabilities across all six vulnerability classes (Prompt Injection, Data Exfiltration, PII/Secrets in Prompts, Insecure Output Handling, Excessive Agency, and Jailbreak Risks). The code is a traditional Express.js route handler with no LLM functionality or data paths.

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


Was this helpful?  👍 Yes  |  👎 No 

@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

This PR adds a test file for fork PR scanning. I reviewed the code changes and found no LLM security vulnerabilities. The code does not involve any LLM interactions, prompt construction, or agent capabilities.

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.

1 participant