Title: Git commit sha detection inconsistent between telemetry and actual operations · Issue #3376 · github/codeql-action · GitHub
Open Graph Title: Git commit sha detection inconsistent between telemetry and actual operations · Issue #3376 · github/codeql-action
X Title: Git commit sha detection inconsistent between telemetry and actual operations · Issue #3376 · github/codeql-action
Description: See: codeql-action/src/status-report.ts Lines 271 to 272 in cac5926 const commitOid = getOptionalInput("sha") || process.env["GITHUB_SHA"] || ""; vs: codeql-action/src/git-utils.ts Lines 48 to 69 in cac5926 export const getCommitOid = as...
Open Graph Description: See: codeql-action/src/status-report.ts Lines 271 to 272 in cac5926 const commitOid = getOptionalInput("sha") || process.env["GITHUB_SHA"] || ""; vs: codeql-action/src/git-utils.ts Lines 48 to 69 i...
X Description: See: codeql-action/src/status-report.ts Lines 271 to 272 in cac5926 const commitOid = getOptionalInput("sha") || process.env["GITHUB_SHA"] || ""; vs: codeql-action/src...
Opengraph URL: https://github.com/github/codeql-action/issues/3376
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Git commit sha detection inconsistent between telemetry and actual operations","articleBody":"See:\n\nhttps://github.com/github/codeql-action/blob/cac5926de5e2c542638262c4e24527972023cdec/src/status-report.ts#L271-L272\n\nvs:\n\nhttps://github.com/github/codeql-action/blob/cac5926de5e2c542638262c4e24527972023cdec/src/git-utils.ts#L48-L69\n\nSo if the checkout is at a different sha from `GITHUB_SHA` then the two are different. In my case this made debugging very difficult because when I reran the workflow with debug logging enabled the telemetry calls were printed and showed me it reporting a different sha from the one that is actually used by the upload action.\n\nI suspect this happened because the the telemetry was added later and we originally implemented the \"sha of current checkout\" behaviour so that it worked outside actions. Removing the \"sha of current checkout\" behaviour, or making it the lowest priority, feels like the \"most correct\" fix, but more risky than just changing the telemetry to use the `getCommitOid` function.","author":{"url":"https://github.com/rneatherway","@type":"Person","name":"rneatherway"},"datePublished":"2025-12-19T10:07:46.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/3376/codeql-action/issues/3376"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:eb32ca8a-a6b1-4cbc-239f-2456e395b357 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | C5C8:367724:110CC2C:1660F54:696B70A2 |
| html-safe-nonce | 1631f4fa3934b8890844469a5580cd77119c18927ce38275200268fa5355c125 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNUM4OjM2NzcyNDoxMTBDQzJDOjE2NjBGNTQ6Njk2QjcwQTIiLCJ2aXNpdG9yX2lkIjoiNjMzMjI1ODc3NTE5NjIwMTEyMiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 32933c20f39bdd25a25d1e8356dfa6df904866c6d36b1388e0623c41a4d5c92c |
| hovercard-subject-tag | issue:3746904048 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/github/codeql-action/3376/issue_layout |
| twitter:image | https://opengraph.githubassets.com/46d3ef4c6f29ccdb39ad28175acd42466003e6d5acb956a61bb50a599ac4d3ce/github/codeql-action/issues/3376 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/46d3ef4c6f29ccdb39ad28175acd42466003e6d5acb956a61bb50a599ac4d3ce/github/codeql-action/issues/3376 |
| og:image:alt | See: codeql-action/src/status-report.ts Lines 271 to 272 in cac5926 const commitOid = getOptionalInput("sha") || process.env["GITHUB_SHA"] || ""; vs: codeql-action/src/git-utils.ts Lines 48 to 69 i... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | rneatherway |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5f99f7c1d70f01da5b93e5ca90303359738944d8ab470e396496262c66e60b8d |
| turbo-cache-control | no-preview |
| go-import | github.com/github/codeql-action git https://github.com/github/codeql-action.git |
| octolytics-dimension-user_id | 9919 |
| octolytics-dimension-user_login | github |
| octolytics-dimension-repository_id | 259445878 |
| octolytics-dimension-repository_nwo | github/codeql-action |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 259445878 |
| octolytics-dimension-repository_network_root_nwo | github/codeql-action |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 82560a55c6b2054555076f46e683151ee28a19bc |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width