Dave Bartolomeo

Results 10 issues of Dave Bartolomeo

This PR adds support for scanning the evaluator log to detect suspicious join orders that may be causing performance problems in a query. The actual detection algorithm is cut-and-pasted, and...

Complexity: Medium

**Is your feature request related to a problem? Please describe.** QL tests in the Test Explorer occasionally go into the "retired" state, where the test result icon is faded. There...

enhancement
VSCode

**DO NOT MERGE** Just for debugging some Actions failures.

Reported by several internal users: ``` [2023-09-06 14:13:16] Evaluation of .../code/ql/java/ql/src/Security/CWE/CWE-022/TaintedPath.ql produced BQRS results. Error running query: Cannot create a string longer than 0x1fffffe8 characters (codeQL.quickEvalContextEditor) Error: Error running query:...

bug
VSCode

Trivial fix is to ignore events for tests we didn't explicitly request in the run. I also discovered that sometimes the CLI emits an empty null-terminated string as an event....

Complexity: Low

This PR updates the new test UI to consume the richer events emitted by `codeql test run` starting in CLI 2.13.1. We now get a notification when a test _starts_...

Complexity: Medium

Resolves #261 Resolves #24 ---- (In draft until I finish implementing tests) ## Behavior ### Before the change? * In both ASP.NET Core and Azure Functions, the webhook secret for...

Note: contains a few IR diffs that I haven't fully investigated. During escape analysis, we currently allow an address to be used as an operand of a `Phi` instruction without...

C++

Changed from `tools: latest` to `tools: linked` to reflect new usage. ### Merge / deployment checklist - [ ] Confirm this change is backwards compatible with existing workflows. - [...