ljharb/qs

Add the ability to enable strict RFC 3986 conformance for decoding as well as encoding

Open

#205 ouverte le 10 avr. 2017

Voir sur GitHub
 (1 commentaire) (5 réactions) (0 assignés)JavaScript (744 forks)batch import
enhancementhelp wantedparsesemver-minor: new features

Métriques du dépôt

Stars
 (8 015 stars)
Métriques de merge PR
 (Merge moyen 6j 22h) (2 PRs mergées en 30 j)

Description

To achieve strict RFC 3986 conformance (specifically, '+' in query strings is a LITERAL '+' and not a placeholder for ' ' as per the W3C's URL recommendations, the following is necessary:

qs.parse(s, {
    decoder: (str) => {
        try {
            return decodeURIComponent(str);
        } catch (e) {
            return str;
        }
    },
})

For those of us who prefer strict RFC 3986 conformance, I suggest an option like:

qs.parse(s, {
    format: 'RFC3986',
})

which would be much more concise and not require knowing the implementation details of the default decode function.

Guide contributeur

Direction de recherche
Implémentez une nouvelle option `format` qui, lorsqu'elle est définie sur `'RFC3986'`, utilise directement `decodeURIComponent` au lieu du décodeur par défaut qui décode `+` comme un espace. Modifiez la fonction parse pour vérifier cette option et appliquer la logique de décodage appropriée.
Stack technique
javascript
Domaine
api
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptURL encoding
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65