Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Incomplete naming schema and API usage patterns in py/insecure-cookie

Ouverte
#21,647 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
52/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
Calme
Stack technique
python
Domaine
security

Piste de recherche

Commencez par la règle py/insecure-cookie et examinez le prédicat cookie.isSensitive ainsi que la gestion de set_cookie décrits dans l’issue. Comparez les noms de tokens listés et les exemples directs de Set-Cookie avec le comportement actuel ; le travail est terminé lorsque les modèles de cookies d’authentification pris en charge sont détectés sans dépendre uniquement des méthodes du framework.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

  1. The existing cookie.isSensitive predicate seems to miss common modern authentication token patterns. I have found some and listed them below.
  2. The rule currently relies primarily on specific framework methods (e.g., set_cookie). It ignores direct HTTP manipulation, which is also very common.

code example:

from flask import Flask, Response, make_response

app = Flask(__name__)

@app.route("/login")
def login():
    resp = make_response("Logged in")
    resp.set_cookie("authKey", "secret123") # $ Alert[py/insecure-cookie]
    resp.set_cookie("accessToken", "secret123") # $ missing
    resp.set_cookie("access_token", "secret123") # missing
    resp.set_cookie("auth_token", "secret123") # missing
    resp.set_cookie("jwt", "secret123") # $ missing
    resp.set_cookie("oauth_token", "secret123") # $ missing


    # cannot support this
    resp.headers.add("Set-Cookie", "authKey=secret123") # missing

    # This is also common, but it seems more difficult to support this.
    from http.cookies import SimpleCookie
    resp = make_response("Logged in")
    cookie = SimpleCookie()
    cookie["session"]['authKey'] = "secret123" # missing
    # cookie["session"]["httponly"] = True
    # cookie["session"]["secure"] = True
    for key, morsel in cookie.items():
        resp.headers.add('Set-Cookie', morsel.OutputString())
    return resp
Langage dominant
CodeQL
Étoiles
10.1k
Forks
2.1k
Merge moyen
2 j 16 h
PR mergées (30 j)
143

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

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

Autres issues de github/codeql

Toutes les issues de github/codeql

Issues similaires

Plus d'issues Security

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.