nodejs/node

fs: utf8 fast paths don't accept all valid utf8 values

Open

#49 888 ouverte le 26 sept. 2023

Voir sur GitHub
 (9 commentaires) (1 réaction) (0 assignés)JavaScript (117 218 stars) (35 535 forks)batch import
fsgood first issue

Description

UTF8 and UTF-8 are also valid but not accepted by any existing utf8 fast paths (#48658, #49884)

Guide contributeur

Stack technique
nodejsjavascriptcpp
Domaine
backendperformance
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
understanding of UTF 8 encodingfamiliarity with Node.js file system internalsbasic C++ debugging skills
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35
Direction de recherche
Investigate the existing utf8 fast paths in Node.js's fs module (likely in lib/internal/fs/utils.js or src/node file.cc). Examine issues #48658 and #49884 for context. Determine where UTF8 and UTF 8 strings are compared and make the comparison case insensitive or add the accepted variants. Ensure consistency with the encoding normalization function. Add tests for the new accepted values.