bug(sdk/go): browser login callback binds 127.0.0.1 but redirect_uri uses localhost
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
- #4531 par @trukhinyuri — fermée sans fusion
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 65/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- go
- Domaine
- authentication
Piste de recherche
Le listener se trouve dans oidc/authcode.go (startCallbackServer, qui appelle net.Listen sur 127.0.0.1) et l'URI de redirection est construite dans oidc/oidc.go (loginBrowser, qui utilise localhost). Lis les deux, puis décide si tu changes l'hôte de redirection en 127.0.0.1 ou si tu fais en sorte que le listener couvre les deux familles de loopback. C'est terminé lorsque l'hôte de l'URI de redirection correspond à ce que le serveur accepte et que le flux Login se termine toujours sur un hôte IPv4 classique. Le problème n'a pas été reproduit : confirme donc le cas IPv6 avant de le considérer comme corrigé.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
User Story
I use OpenShell and build automation against its SDKs. I found this while reviewing sdk/ on main; it is identified from the code paths cited below and has not yet been reproduced against a live gateway.
Problem Statement
startCallbackServer listens on 127.0.0.1:<port>, but loginBrowser sends redirect_uri=http://localhost:<port>/callback. The CLI uses 127.0.0.1 for both.
Impact / Why This Matters
On hosts or browsers where localhost resolves to ::1 first without IPv4 fallback, the IdP redirect fails to reach the callback and Login times out.
Acceptance Criteria
- The redirect URI host matches the address the callback server listens on (or the server listens on both loopback families).
Reproduction Steps
Code reference: oidc/authcode.go net.Listen("tcp", "127.0.0.1:%d") vs oidc/oidc.go fmt.Sprintf("http://localhost:%d/callback", port). On a host where localhost maps only to ::1, run oidc.Login.
Suggested UX (if applicable)
No response
Environment
- OpenShell:
main@ 82977e2 (2026-10-13) - SDK: Go (
sdk/go/openshell/v1/oidc) - Toolchain: Go 1.25.4 linux/amd64, Node 22
Logs
- Langage dominant
- Rust
- Étoiles
- 15.4k
- Forks
- 1.7k
- Merge moyen
- 2 j 6 h
- PR mergées (30 j)
- 397
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de NVIDIA/OpenShell
-
state:triage-needed
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
bug(sdk/go): edge (Cloudflare Access) token is cached for the client lifetime and never reloadedOuvertestate:triage-needed
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
NVIDIA/OpenShell#4515 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
state:triage-needed
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
NVIDIA/OpenShell#4509 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
state:triage-needed
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
docs(upgrade): no migration notes for 0.1.1, 0.1.2, or 0.1.3 (only the 0.1.0 page exists)Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertestate:triage-needed
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de NVIDIA/OpenShell
Issues similaires
-
ai_p1 comp-message-queues
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
ClickHouse/ClickHouse#125890 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
storytold/photocraft#5710 ·
Les mainteneurs répondent en général sous 1 jour
-
Two tests fail on a Windows checkout: a CRLF parse in feature_walk_tests, an 8.3 TEMP path in libraries_testsPeut-être pris @CamWight l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
bherbruck/solvecraft#1186 · 1 commentaire ·
-
area: screenshot audit bug good first issue priority: P3 security severity: info type: correctness
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
pierruno-labs/prowl#31 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
limitlessprojects/Go-Hybrid#50 ·
Les mainteneurs répondent en général sous 1 jour