nodeSolidServer / nodeSolidServer/node-solid-server

issuer URIs with a path part don't work

Offen
#1,265 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

bug
Vorherrschende Sprache
JavaScript
Sterne
1.8k
Forks
308
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

i originally opened this as https://github.com/solid/oidc-auth-manager/issues/44 but perhaps i should have started here to target the right audience.

oidc-auth-mananger seems to make an invalid assumption that all OIDC Issuers are URIs with an empty path part. if a webid has an OIDC Issuer URI with a path part (like my webid https://zenomt.zenomt.com/card.ttl#me with issuer https://zenomt.com/oidc/), it can never be matched against the id_token iss claim because the discovered issuer URI is reduced to its origin before being compared. see these lines in preferred-provider.js.

OIDC Issuers are allowed to have path parts.

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

Beginnen Sie in src/preferred-provider.js bei den Zeilen 70–73, wo die erkannte Issuer-URI vor dem Vergleich mit dem id_token iss-Claim verkürzt wird. Überprüfen Sie die OpenID Connect-Anforderung für Pfadteile des Issuers und bewahren Sie den Pfad beim Abgleich. Erledigt ist die Aufgabe, wenn ein Issuer wie https://zenomt.com/oidc/ mit dem entsprechenden iss-Claim übereinstimmen kann.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
authentication
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

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