simplesamlphp / simplesamlphp/simplesamlphp-module-oidc
Feature request: allow to specify the client_id
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 50
- Forks
- 28
- Ø Merge
- 1 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
When registering a new client, the client_id is auto-generated and not very human-read/write friendly.
There are use cases where users are expected to provide the client_id themselves in UI. Example: Thunderbird Beta provides a "Custom OAuth Configuration" dialog, see below.
I already commented to Mozilla regarding the use of one .well-known URL instead of 3 to 4 free-text fields ( https://bugzilla.mozilla.org/show_bug.cgi?id=1602166#c58 ) but what remains is the client_id as an extra hurdle.
For a public client, being able to choose a user-friendly string such as "MozillaThunderbird" would reduce error-proneness of OAuth2 configuration there.
The suggestion is thus to enrich the Client Registration UI in the SSP admin portal such that a client_id is auto-generated, but make it a suggestion end changeable in UI.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne bei der Client Registration UI im SSP-Adminportal und verfolge, wie die client_id generiert und übermittelt wird. Prüfe den bestehenden Registrierungsablauf und dessen Tests, sofern vorhanden. Als abgeschlossen gilt die Aufgabe, wenn eine generierte client_id als bearbeitbarer Vorschlag angezeigt wird und ein vom Benutzer angegebener Wert für einen neuen Public Client akzeptiert wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- authentication
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 65/100