Skip to content

fix(security): address CodeQL scanning alerts#48

Merged
Jaganpro merged 1 commit into
mainfrom
fix/code-scanning-alerts
May 4, 2026
Merged

fix(security): address CodeQL scanning alerts#48
Jaganpro merged 1 commit into
mainfrom
fix/code-scanning-alerts

Conversation

@Jaganpro

@Jaganpro Jaganpro commented May 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Exclude the generated Agent Script SDK bundle from CodeQL analysis instead of hand-editing vendored output
  • Replace shell-string process execution with argv-based execution in maintenance scripts
  • Harden URL/header handling and remove file existence pre-check patterns flagged by CodeQL
  • Add narrow CodeQL suppressions for intentional temp-file spillover/download flows with inline justification
  • Add regression coverage for Salesforce org URL parsing and unsafe cached ETag handling

Validation

  • npm run validate
  • npm run eslint

Notes

Local CodeQL CLI is not installed, so this PR relies on the GitHub CodeQL workflow for final scanner verification.

@Jaganpro Jaganpro force-pushed the fix/code-scanning-alerts branch from c981604 to 42d394d Compare May 4, 2026 15:47
@Jaganpro Jaganpro force-pushed the fix/code-scanning-alerts branch from 42d394d to 9b4cb98 Compare May 4, 2026 15:50
@Jaganpro Jaganpro merged commit 308800c into main May 4, 2026
15 checks passed
@Jaganpro Jaganpro deleted the fix/code-scanning-alerts branch May 4, 2026 16:17
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