Skip to content

Create test.js#1

Open
tornidomaroc-web wants to merge 1 commit into
mainfrom
tornidomaroc-web-patch-1
Open

Create test.js#1
tornidomaroc-web wants to merge 1 commit into
mainfrom
tornidomaroc-web-patch-1

Conversation

@tornidomaroc-web

Copy link
Copy Markdown
Owner

No description provided.

@vercel

vercel Bot commented Apr 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
scan-and-action Ready Ready Preview, Comment Apr 4, 2026 9:51pm

@tornidomaroc-web

Copy link
Copy Markdown
Owner Author

Fixor · SQL injection report

Repository: tornidomaroc-web/scan-and-action · PR: #1
Scan ID: demo-validate-1775341989546 · Commit: 8e2bbd8a342ae9e3438c0dd3960ff1329aac7229

Summary

Workflow status success
Automation ready true
Automation note All patches high/medium quality and no warnings
Semgrep findings (total) 2
SQL injection (classified) 1
Skipped (other rules) 1
Fixes generated 1
Patch quality high: 1 · medium: 0 · low: 0
Duration 2 ms

Suggested fixes

Expand a row for original → suggested code, warnings, and explanation.

1. `src/routes/reports.js:18` · **high** · `SQL_INJECTION`
  • Dialect: mysql · Detection confidence: high
  • Parameter expressions: merchantId, dayStr

Original

  return pool.query('SELECT total FROM payouts WHERE merchant_id = ' + merchantId + ' AND day = ' + dayStr);

Suggested

return pool.query('SELECT total FROM payouts WHERE merchant_id = ? AND day = ?');

Explanation

Replaced dynamic string concatenation with placeholders. Pass parameterValues in order as the driver's bound-parameter array (second argument to query).

Generated by Fixor · finished at 2026-04-04T22:33:09.549Z

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