Git commit sha detection inconsistent between telemetry and actual operations

Ouverte

#3,376 ouverte le 19/12/2025 2 commentaires 0 réactions 1 personne assignée Voir sur GitHub

@mbg y travaille déjà.

Depuis le 19/12/2025.

Évaluation

Cette issue n'a pas encore été évaluée.

Description

See:

https://github.com/github/codeql-action/blob/cac5926de5e2c542638262c4e24527972023cdec/src/status-report.ts#L271-L272

vs:

https://github.com/github/codeql-action/blob/cac5926de5e2c542638262c4e24527972023cdec/src/git-utils.ts#L48-L69

So 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.

I 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.

Langage dominant: TypeScript
Étoiles: 1.6k
Forks: 493
Merge moyen: 1 j 13 h
PR mergées (30 j): 44

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

Lisez l'issue en entier, puis le guide de contribution du projet.
Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
Forkez le dépôt et travaillez sur une branche.
Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de github/codeql-action

upload-sarif always warns when setting partialFingerprints Ouverte

Difficulté 2/5 1-3 heures Accessibilité débutants 68/100

github/codeql-action#4052 · 4 commentaires · Ouverte le 29/7/2026
Ruby (build-mode=none) default-setup analysis fails with "no source code was seen" on pull_request events, but succeeds on push — traced to --no-calculate-baseline Ouverte

Difficulté 4/5 3-5 jours Accessibilité débutants 45/100

github/codeql-action#4078 · 1 commentaire · Ouverte le 5/8/2026
CodeQL autobuild hangs forever on failure Ouverte

Difficulté 4/5 3-5 jours Accessibilité débutants 48/100

github/codeql-action#4008 · 9 commentaires · Ouverte le 14/7/2026
`codeql-action/init` Makes Metal Toolchain Unavailable on macOS 26 Runners Ouverte

Difficulté 4/5 3-5 jours Accessibilité débutants 52/100

github/codeql-action#3978 · 4 commentaires · 1 réaction · Ouverte le 26/6/2026
CodeQL fails to detect rust/insecure-cookie in Rust code Ouverte

Difficulté 3/5 1-2 jours Accessibilité débutants 48/100

github/codeql-action#3915 · 6 commentaires · 3 réactions · Ouverte le 19/5/2026

Toutes les issues de github/codeql-action

Issues similaires

ls -i / --inode not supported ("invalid option -- i"), breaks autoconf's `ls -di .` working-directory check Ouverte

Difficulté 2/5 1-3 heures Accessibilité débutants 75/100

vercel-labs/just-bash#464 · Ouverte le 24/9/2026
looksLikeSlug() is ASCII-only, so non-Latin entity slugs (e.g. Korean) skip exact match and collapse Ouverte

Difficulté 2/5 1-3 heures Accessibilité débutants 75/100

garrytan/gbrain#5421 · Ouverte le 24/9/2026
Passer le formulaire de publication en BAL 1.5 par défaut Ouverte

Difficulté 2/5 1-3 heures Accessibilité débutants 65/100

BaseAdresseNationale/adresse.data.gouv.fr#2583 · Ouverte le 24/9/2026
🎙️ task - fix(tests-node): the shipped catch ships cause error where error is unknown Ouverte

Difficulté 2/5 1-3 heures Accessibilité débutants 65/100

ehmpathy/declapract-typescript-ehmpathy#632 · Ouverte le 24/9/2026
Search modal renders beneath the open AI dock (shared --z-overlay) Ouverte

Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100

TanStack/tanstack.com#1293 · Ouverte le 24/9/2026

Plus d'issues TypeScript