Improve broken security-events handling for private repositories without Advanced Security

Offen

#2,125 erstellt am 08.2.2024 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 50/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Ruhig
Tech-Stack: github-actions, typescript
Bereich: ci-cd, security

Rechercherichtung

Beginne mit der Diskussion zu Pull Request 2121 und Issue 2117 und vergleiche anschließend die Abschnitte von check-spelling's unknown-words.sh in den Zeilen 1122-1126 und 1610-1618. Verfolge die Verarbeitung von security-events/SARIF durch die Action und verifiziere, dass private Repositories ohne Advanced Security eine nicht-fatale Antwort erhalten, statt laufende aufrufende Workflows zu beenden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

As noted in https://github.com/github/codeql-action/pull/2121#discussion_r1483012019 and https://github.com/github/codeql-action/issues/2117#issuecomment-1934863805, check-spelling has special code to handle the case where a user thinks they can use security-events: ... to talk to sarif reporting, but they're in a private repository and that repository doesn't have GitHub Advanced Security enabled:

https://github.com/check-spelling/check-spelling/blob/26b46adbdebd5dd0b34c7155113d50c40f43fb22/unknown-words.sh#L1610-L1618
https://github.com/check-spelling/check-spelling/blob/26b46adbdebd5dd0b34c7155113d50c40f43fb22/unknown-words.sh#L1122-L1126

Which results in this (more or less):

Check Spelling: .github/workflows/spelling.yml#L106
Unsupported configuration: use_sarif needs GitHub Advanced Security to be enabled - see https://docs.github.com/get-started/learning-about-github/about-github-advanced-security. (unsupported-configuration)

The codeql-action should have some code to handle this as well.

Ideally it'd do it in a way that is not fatal to actions/workflows that consume it.

check-spelling has to carefully decide not to call github/codeql-action today if it figures out that it'll fail, otherwise the action/workflow die because of this fault.

Vorherrschende Sprache: TypeScript
Sterne: 1.6k
Forks: 493
Ø Merge: 1 T. 13 Std.
Gemergte PRs (30 T.): 44

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus github/codeql-action

upload-sarif always warns when setting partialFingerprints Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

github/codeql-action#4052 · 4 Kommentare · Erstellt am 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 Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100

github/codeql-action#4078 · 1 Kommentar · Erstellt am 05.8.2026
CodeQL autobuild hangs forever on failure Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

github/codeql-action#4008 · 9 Kommentare · Erstellt am 14.7.2026
`codeql-action/init` Makes Metal Toolchain Unavailable on macOS 26 Runners Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 52/100

github/codeql-action#3978 · 4 Kommentare · 1 Reaktion · Erstellt am 26.6.2026
CodeQL fails to detect rust/insecure-cookie in Rust code Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100

github/codeql-action#3915 · 6 Kommentare · 3 Reaktionen · Erstellt am 19.5.2026

Alle Issues in github/codeql-action