microsoft / microsoft/libHttpClient

Redirects should not pass along the authorization header for security and usability reasons

Offen
#547 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
C++
Sterne
295
Forks
144
Ø Merge
2 T. 19 Std.
Gemergte PRs (30 T.)
2

Beschreibung

Apple does this by default (see https://github.com/Alamofire/Alamofire/issues/798) but it appears the Windows http implementation give flexibility here. To be consistent across platforms as well not leak auth tokens to other locations, the auth header should be removed.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die Behandlung von Weiterleitungen in der Windows-HTTP-Implementierung nachzuverfolgen, und vergleiche sie anhand des verlinkten Alamofire-Issues mit dem Standardverhalten von Apple. Stelle fest, wie Autorisierungs-Header über Weiterleitungen hinweg mitgeführt werden, und stelle anschließend sicher, dass weitergeleitete Requests den Autorisierungs-Header nicht an andere Ziele übermitteln, und überprüfe das plattformkonsistente Verhalten mit den bestehenden Checks des Projekts.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
networking, security
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
32/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.