Création d'un parser de querystring

Bonjour,

Pour une application dans un site web, j’ai besoin de récupérer le Token qui se trouve être en document.location.hash sous le format d’une querystring.

C’est le passage clé de mon site si je m’y prends mal, l’application ne fonctionnera pas. Je voulais savoir si vous remarquiez un potentiel oubli dans mon parsing ou un format que je ne supporte pas.

Pour test : https://jsfiddle.net/v2w18tao/79/

function querystringParser(hash, opt) {
  var set = function (defVal, optVal) {
    return (typeof optVal === "undefined") ? defVal : optVal;
  };

  var options = {
    starter: set("?", opt && opt.starter),
    sep: set("&", opt && opt.sep),
    eq: set("=", opt && opt.eq)
  };
  var obj = {},
    enableStarter = (typeof options.starter === "string");
  for (var i = 0, varName = null, val = null, char, key; i <= hash.length; i++) {
    char = hash[i];
    if (i === 0 && char === options.starter && enableStarter) { // Enlève le "#"
      continue;
    } else if (val === null && char === options.eq) { // On passe à la suite si "="
      val = "";
    } else if (val === null) {
      varName = ((varName === null) ? "" : varName) + char;
    } else if (varName !== null && char !== options.sep && i !== hash.length) {
      val = val + char;
    } else {
      if (varName !== "") {
        key = decodeURIComponent(varName);
        switch (typeof obj[key]) {
          case "undefined":
            obj[key] = decodeURIComponent(val);
            break;
          case "string":
            obj[key] = [decodeURIComponent(val)];
          default:
            obj[key].push(decodeURIComponent(val));
        }
      }

      varName = null;
      val = null;
    }
  }
  return obj;
}

var querystring = "#token=a567082ca54abf223a77da38&scope=user%3Aread+channel%3Amoderate&token_type=auth2&token_type=auth";
document.getElementById("json").textContent = JSON.stringify(querystringParser(querystring, {starter:"#"}), null, 4);

Pour le côté apprentissage, je souhaiterais mettre ma fonction sur github (sous forme de bibliothèque par exemple) et faire automatiquement des tests quand je PR (comme le zds) via : https://qunitjs.com/ est-ce possible ? Dois-je changer de lib ? Faut-il des connaissances avancées ?

J’ai aussi pensé à faire un script node pour tester le résultat.

Bon vol,

A.

EDIT : Rajout des tags.

Pourquoi tu fais ça à la main ?

On peut le faire avec une fonction native dans le navigateur ?

Disons qu’il y a déjà des tas de fonctions mieux que celle que tu montres pour faire ça. En revanche ce qu’elles font c’est parser une query string, pas parser une query string seulement si elle préfixée par #.

Du coup conseils :

• Une fonction qui fait UN truc c’est mieux qu’une fonction qui en fait plusieurs. Découpe ça en 2 fonctions : une qui extrait la query string, l’autre qui la parse. Et la 2e t’as pas besoin de l’écrire, t’en trouveras des tas de très bien partout, t’as même donné un exemple.
• Evite de réinventer la roue.
• Dans un topic de ce genre, ça vaut la peine d’expliquer pourquoi tu te retrouves à devoir parser une query string passée aussi bizarrement que ça. Si ça se trouve c’est un hack moche parce que t’as raté la bonne solution.

L’API avec authentication oauth me renvoi sur : monsite.com/callback#token=a567082ca54abf223a77da38&scope=user%3Aread+channel%3Amoderate&token_type=auth2&token_type=auth, je ne peux donc pas utiliser querystring de node.

Pourquoi est-ce qu’elle te redirige sur cette URL ? Le principe d’OAuth c’est pas que tu dis au provider vers quelle URL rediriger le client ? Si oui, pourquoi rediriger vers example.com/foo# au lieu d'example.com/foo? (Le ? s’applique à l’URL, mais si tu préfères tu peux l’appliquer à la question.)

J’ai choisi l'OAuth 2.0 Implicit Grant pour une question de simplicité et le fait d’être léger pour le serveur. Le serveur me sert uniquement à distribuer des pages statiques avec ce mode.