Code does not check for session = null when obtaining webId
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 153
- Forks
- 46
- Merge moyen
- 1 j 8 h
- PR mergées (30 j)
- 7
Description
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
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
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.
Piste de recherche
Commencez dans src/signin.js, autour du gestionnaire signInPopUpButton vers la ligne 739, puis comparez sa gestion de session avec webIdFromSession vers la ligne 783. Reproduisez le flux de connexion avec un domaine personnalisé lorsque la session est fermée et vérifiez qu’une session null ne provoque plus le TypeError webId signalé et que le flux attendu de connexion au fournisseur et de bearer-token peut se poursuivre.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- authentication, frontend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 35/100