Code does not check for session = null when obtaining webId
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 153
- Forks
- 46
- Ø Merge
- 1 T. 8 Std.
- Gemergte PRs (30 T.)
- 7
Beschreibung
How to reproduce
- Logged out and cleared localStorage
- Attempted to login to https://melvin.solid.live/public/RWWCrew/
- Click on custom domain, https://melvincarvalho.com/#me
Observed behaviour
Uncaught (in promise) TypeError: Cannot read property 'webId' of null
at signin.js:732
at <anonymous>
in
signInPopUpButton.addEventListener('click', function () {
var offline = offlineTestID();
if (offline) return setUserCallback(offline.uri);
return solidAuthClient.popupLogin({ popupUri: $SOLID_GLOBAL_config.popupUri }).then(function (session) {
var webIdURI = session.webId;
Expected Behaviour
What I would expect is to redirect me to my oidc provider, let me log in, and then give me a bearer token
Possible fix
https://github.com/solid/solid-ui/blob/master/src/signin.js#L739 should use webIdFromSession
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 src/signin.js ungefähr beim Handler signInPopUpButton in der Nähe von Zeile 739 und vergleiche anschließend dessen Sitzungsbehandlung mit webIdFromSession in der Nähe von Zeile 783. Reproduziere den Login-Ablauf für eine benutzerdefinierte Domain im abgemeldeten Zustand und überprüfe, dass eine null-Sitzung nicht mehr den gemeldeten webId TypeError verursacht und dass der erwartete Provider-Login- und Bearer-Token-Ablauf fortgesetzt werden kann.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- authentication, frontend
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100