Regression v1.3.0+ in Safari

Abierto
  • ¿Algo no está bien?
#681 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
javascript
Área
tooling

Línea de trabajo

Reproduce el fallo en el ejemplo docusaurus-openapi con yarn install, yarn build-packages y yarn start, y compara después el resultado del lockfile de v1.1.5 con v1.3.0–v1.5.0. Inspecciona los cambios en las expresiones regulares asociados con PR 651 de postman-code-generators y la ruta que sigue la función convert. Se considera terminado cuando Safari carga sin la excepción, mientras el comportamiento de Chrome permanece sin cambios.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Describe the bug

Using this package at ^1.3.0 results in the following exception in Safari:

Unhandled Promise Rejection: SyntaxError: Invalid regular expression: invalid group specifier named

This will crash any implementation that is not try/catching their usage around this library (e.g. when using the convert function).

To Reproduce
Steps to reproduce the behavior:

You can reproduce this issue by cloning this repository: https://github.com/cloud-annotations/docusaurus-openapi

  1. yarn install (note: the lock file is currently resolving 1.1.5)
  2. yarn build-packages
  3. yarn start
  4. In Safari, navigate to: http://localhost:3000/petstore
  5. Observe: The page loads, no errors in the console
  6. In Chrome, navigate to: http://localhost:3000/petstore
  7. Observe: The page loads, no errors in the console

Now, delete the yarn.lock file at the root of the repository and repeat the steps. (this will resolve 1.5.0)

  • In Safari, the page will not load and the following error will be displayed in the console:

Unhandled Promise Rejection: SyntaxError: Invalid regular expression: invalid group specifier named

  • In Chrome, the page will load and there is no error in the console.

Expected code snippet and corresponding request

N/A

Screenshots

N/A

Additional context

I tried to narrow which version was causing this issue and it appears to be in the v1.3.0 release era (happens in v1.4.0 and v1.5.0 as well).

It could be from this PR: https://github.com/postmanlabs/postman-code-generators/pull/651, however the changelog doesn't currently link to commits and is somewhat hard to narrow changes per release.

Safari does not support look behind with regular expression matching, so whatever change in v1.3.0 introduces that usage, is likely the culprit.

Lenguaje dominante
JavaScript
Estrellas
1k
Forks
382
Sin PR fusionados en 30 d

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de postmanlabs/postman-code-generators

Todos los issues de postmanlabs/postman-code-generators

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.