ExecJS.eval does not tolerate trailing semicolon
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 42/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript, ruby
- Domaine
- backend
Piste de recherche
Start at the ExecJS.eval entry point and trace the MiniRacerRuntime path for input ending in a semicolon. Confirm the behavior with the example '1+2;' and add regression coverage showing that it evaluates successfully as 3 rather than raising the reported SyntaxError.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
irb(main):009:0> ExecJS.runtime
=> #<ExecJS::MiniRacerRuntime:0x007feb4abf7178>
irb(main):010:0> ExecJS.eval '1+2'
=> 3
irb(main):011:0> ExecJS.eval '1+2;'
ExecJS::RuntimeError: Uncaught SyntaxError: Unexpected token ; at undefined:1:4
I think ExecJS.eval should tolerate trailing semicolons. I had this happen in a library stack where a library js file ended with a semicolon and the stack was feeding it to eval.
- Langage dominant
- Ruby
- Étoiles
- 578
- Forks
- 285
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de rails/execjs
-
Runtime stderr is missing from ruby error messagePeut-être pris @arpitjain099 l’a pris il y a 141 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
Toutes les issues de rails/execjs
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
up-for-grabs/up-for-grabs.net#6330 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 1-3 heures Accessibilité débutants 72/100
TheOdinProject/curriculum#31490 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
hotwired/turbo-rails#791 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
FinanceFlash/unvibecode#273 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour