Skip to content

probe: CodeQL on the alert 17 fix - #61

Closed
DarrellVS wants to merge 1 commit into
mainfrom
codeql-17-typeof
Closed

DarrellVS wants to merge 1 commit into
mainfrom
codeql-17-typeof

Conversation

@DarrellVS

Copy link
Copy Markdown
Owner

Probe for CodeQL only: code scanning analyses pull requests into main, and this checks the alert 17 fix before 3.5.2. Closed once CodeQL reports; the release goes through the usual fast-forward.

🤖 Generated with Claude Code

Code scanning still reported alert 17 on 3.5.1, at the call rather than the
lookup: with a key chosen by the request, it wants the value checked as
callable first. The Map from 3.5.1 already limits it to the listed routes;
this adds the typeof guard the rule recognises.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@DarrellVS

Copy link
Copy Markdown
Owner Author

CodeQL passes with no open alerts on this branch, so the fix clears alert 17. Going in through dev; ships as 3.5.2.

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