Test for trialing slashes in value for 'issuer' from '/.well-known/openid-configuration' responses

Offen

#1,511 erstellt am 09.11.2020 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: javascript
Bereich: authentication

Rechercherichtung

Beginne damit, den Code zu lokalisieren, der die URL /.well-known/openid-configuration aus dem vom OpenID-Konfigurations-Response zurückgegebenen Issuer erstellt. Füge Abdeckung für Issuer-Werte mit und ohne abschließenden Schrägstrich hinzu und bestätige, dass beide eine gültige URL mit genau einem Schrägstrich anstelle einer ungültigen JSON-Antwort erzeugen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Basically our ESS Brokers are advertising themselves (in their /.well-known/openid-configuration responses) as issuers with a trailing slash, e.g. for https://broker.pod.inrupt.com/.well-known/openid-configuration:

{
    "introspection_endpoint": "https://broker.pod.inrupt.com/introspect",
    "scopes_supported": [
        "openid",
        "offline_access",
        "webid"
    ],
    "issuer": "https://broker.pod.inrupt.com/",
    :

Note the trailing slash on the issuer URL (which is technically (slightly!) more correct than having the URL without the trailing slash).
So it seems NSS might be blindly taking that issuer value, and appending the string literal /.well-known/openid-configuration to it, without first checking if the issuer value has a trailing slash or not. If the URL constructed has two slashes, then the server responds with it’s standard HTML ‘Page Not Found’ response, which results in the ‘invalid json response…’ error.

If this is correct, then the fix in NSS should be simply checking for a trailing slash before appending /.well-known/openid-configuration.

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

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

Mehr aus nodeSolidServer/node-solid-server

Disclaimer in README.md about non-production readiness Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100

nodeSolidServer/node-solid-server#1848 · 2 Kommentare · 1 Reaktion · Erstellt am 07.2.2026
Enable WebID-TLS integration tests (fix self-signed cert verification loop) Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

nodeSolidServer/node-solid-server#1841 · 2 Kommentare · Erstellt am 08.1.2026
Delete account must point to POD provider Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

nodeSolidServer/node-solid-server#1147 · Erstellt am 15.3.2019
remove bootstrap dependency Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

nodeSolidServer/node-solid-server#1867 · Erstellt am 09.5.2026
Align NSS with LWS 1.0 Authentication Suite Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100

nodeSolidServer/node-solid-server#1853 · Erstellt am 23.4.2026

Alle Issues in nodeSolidServer/node-solid-server