github / github/codeql

Incomplete documentation for cs/web/broad-cookie-domain

Ouverte
#15,169 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
C# documentation
Langage dominant
CodeQL
Étoiles
10.1k
Forks
2.1k
Merge moyen
2 j 15 h
PR mergées (30 j)
141

Description

I believe the CodeQL documentation here conflicts with MDN by omission:
> In this example cookie1 is accessible from online-bank.com ...
> ```
> HttpCookie cookie1 = new HttpCookie("sessionID");
> cookie1.Domain = "online-bank.com";
> ```

https://codeql.github.com/codeql-query-help/csharp/cs-web-broad-cookie-domain/

According to MDN, that cookie would _also_ be available to subdomains:
> For example, if you set Domain=mozilla.org, cookies are available on mozilla.org and its subdomains like developer.mozilla.org.

https://developer.mozilla.org/en-US/docs/Web/HTTP/Cookies#domain_attribute

The CodeQL documentation should state that "In this example cookie1 is accessible from online-bank.com _and its subdomains_".

Guide de contribution

Ouvrir le guide de contribution

Évaluation

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

Recevez les nouvelles issues par e-mail

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