Skip to content

Commit a181906

Browse files
committed
fix(github): run the two-phase PR comment on the secure transport
The file-comment flow posted its comment from `transformResponse` with a bare global `fetch`, so that request carried no abort signal, no response ceiling and no DNS/SSRF validation — cancelling a workflow still left the comment posted. `transformResponse` cannot receive the signal; `directExecution` can. Both tools now run the lookup and the POST through a new `secureGitHubRequest`, mirroring `secureBitbucketRead`, with the signal forwarded to each. Routing, line coercion and the `errors[]` detail are unchanged, and a failed response still throws an error carrying `status`/`statusText`/`data` as the transport does. Request bodies and the comment payload are explicitly typed instead of `Record<string, any>`.
1 parent 00ff23b commit a181906

4 files changed

Lines changed: 474 additions & 195 deletions

File tree

0 commit comments

Comments
 (0)