nodejs/node

/etc/localtime symlink handling

Open

#26 880 ouverte le 23 mars 2019

Voir sur GitHub
 (17 commentaires) (0 réactions) (0 assignés)JavaScript (35 535 forks)batch import
help wantedi18n-apistale

Métriques du dépôt

Stars
 (117 218 stars)
Métriques de merge PR
 (Merge moyen 18j 17h) (219 PRs mergées en 30 j)

Description

  • Version: v11.7.0
  • Platform: Linux gallium 4.19.25 1-NixOS SMP PREEMPT Sat Feb 23 08:07:27 UTC 2019 x86_64 GNU/Linux
  • Subsystem:

Node has trouble figuring out the timezone if /etc/localtime is an indirect symlink. Here's some examples in the REPL

> Date()
'Sun Mar 24 2019 01:42:40 GMT+1000 (GMT+10:00)'    # Wrong, it's GMT+11 in Sydney right now
> new Date().toLocaleString()
'3/24/2019, 1:42:55 AM'                            # Similarly to above
> Intl.DateTimeFormat().resolvedOptions().timeZone
undefined

Even worse, in electron:

> new Date().toLocaleString()
VM575:1 Uncaught RangeError: Unsupported time zone specified undefined
    at new DateTimeFormat (native)
    at Date.toLocaleString (native)
    at <anonymous>:1:12

This is the cause of FreeTubeApp/FreeTube#233. It might also be the cause of ValveSoftware/steam-for-linux#5609 and flathub/im.riot.Riot#32.

Guide contributeur

Direction de recherche
Étudiez comment Node.js résout le fuseau horaire à partir de /etc/localtime, probablement dans src/node time.cc ou similaire. Le problème mentionne des liens symboliques indirects ; vérifiez le code qui lit le lien symbolique et le suit jusqu'au fichier de fuseau horaire réel. Consultez le problème lié FreeTubeApp/FreeTube#233 pour le contexte. Envisagez d'utiliser realpath() ou similaire pour résoudre complètement les liens symboliques avant l'analyse. Si la correction implique JavaScript, elle pourrait être dans lib/internal/process.js ou lib/util.js.
Stack technique
javascriptnodejs
Domaine
backend
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40