Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

Auth transports add credentials after cross-origin redirects

Offen
  • Stimmt etwas nicht?
#4,365 8 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Ein zugehöriger Pull Request wurde bereits gemerged.

  • #4364 von @huynhtrungcsc — ohne Merge geschlossen
  • #4564 von @gmlewis — gemerged

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
25/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
go
Bereich
backend-api-design, security

Rechercherichtung

Beginne mit BasicAuthTransport und UnauthenticatedRateLimitedTransport, insbesondere mit ihren RoundTrip-Methoden, und reproduziere das Verhalten mit zwei httptest-Servern wie beschrieben. Fertig ist die Aufgabe, wenn eine Regressionstestabdeckung für beide Transports vorhanden ist und umgeleitete Anfragen an einen anderen Ursprung nicht ihre von den Transports hinzugefügten Autorisierungsheader enthalten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Summary

BasicAuthTransport and UnauthenticatedRateLimitedTransport add credentials in RoundTrip for each outgoing request. When Go's net/http client follows a redirect to a different origin, the redirected request can pass through the same transport and receive those credentials again.

Expected behavior

Credentials added by these transports should only be sent to the original origin. Redirected requests that cross origin should continue without these transport-managed credentials.

Reproduction

This can be reproduced with two httptest servers: one server returns a redirect to a second server, and the second server records whether the redirected request contains the transport-added authorization headers.

Proposed fix

PR #4364 avoids adding these credentials on cross-origin redirect requests and includes regression coverage for both affected transports.

Vorherrschende Sprache
Go
Sterne
11.3k
Forks
2.6k
Ø Merge
2 T. 10 Std.
Gemergte PRs (30 T.)
35

Entwicklungsumgebung

Erste Schritte

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

Mehr aus google/go-github

Alle Issues in google/go-github

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

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