False positive: "Missing cross-site request forgery token validation" in C# API for endpoint with no cookie/session tokens

Offen

#21,665 erstellt am 08.4.2026 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

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

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Ruhig
Tech-Stack: csharp
Bereich: api, security

Rechercherichtung

Beginne mit der Durchsicht von Commit 5bb31afc834f53d5ea719d782744ff9c7ab70fc0 und dem C# API-Beispiel in diesem Issue, um zu verstehen, warum der Befund zutrifft. Finde den Einstiegspunkt der Abfrage und die vorhandenen Tests dazu und verifiziere anschließend, dass reine API-Key-Server-to-Server-Endpunkte nicht markiert werden, während Endpunkte mit Cookie- oder Token-Authentifizierung weiterhin markiert werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

false-positive

Description of the false positive

We are using a .NET 10 Api and since commit https://github.com/github/codeql/commit/5bb31afc834f53d5ea719d782744ff9c7ab70fc2 we get a false positive on a server -> server endpoint where we have a ApiKey in the header with no tokens or cookies.

Is it possible to extend the check to look if cookie/token authentication is used instead of a ApiKey?

Code samples or links to source code

public class Controller()
{

    [HttpPost()]
    public string Execute()
    {
        return "test";
    }

Vorherrschende Sprache: CodeQL
Sterne: 10.1k
Forks: 2.1k
Ø Merge: 2 T. 16 Std.
Gemergte PRs (30 T.): 143

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

[aw] Update rust-analyzer has stale lock file Offen

agentic-workflows

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

github/codeql#22656 · Erstellt am 23.9.2026
JavaScript externs model StorageEvent() with no type argument Offen

false-positive javascript

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

github/codeql#22632 · 1 Kommentar · Erstellt am 20.9.2026
C#: cs/simplifiable-boolean-expression false positive on Nullable<bool> compared with a literal Offen

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

github/codeql#22556 · Erstellt am 14.9.2026
Add AlertSuppression.ql for Rust (inline // codeql[...] suppression) Offen

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

github/codeql#21637 · 2 Kommentare · Erstellt am 02.4.2026
Organization owned immutable actions raise warning about unpinned 3rd party actions tag Offen

false-positive

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

github/codeql#21076 · 3 Kommentare · 3 Reaktionen · Erstellt am 21.12.2025

Alle Issues in github/codeql