SolidOS / SolidOS/solid-ui

Code does not check for session = null when obtaining webId

Ouverte
#32 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

authentication input-needed
Langage dominant
TypeScript
Étoiles
153
Forks
46
Merge moyen
1 j 8 h
PR mergées (30 j)
7

Description

How to reproduce

  1. Logged out and cleared localStorage
  2. Attempted to login to https://melvin.solid.live/public/RWWCrew/
  3. 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

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. 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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.