evanw / evanw/node-source-map-support

Why use Buffer as an implicit dependency?

Ouverte
#95 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
JavaScript
Étoiles
2.2k
Forks
223
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

☝️ 👉 https://github.com/evanw/node-source-map-support/blob/6296f616ca56fc1e1ced36ea15a2eb43d4227c54/source-map-support.js#L111

Background:
We (https://github.com/CartoDB/cartodb) suffered a Uncaught RangeError: Maximum call stack size exceeded exception on uncaught exceptions after upgrading browserify. I found some similar reported stuff here, e.g. #86 and #93, but after some investigation I found the culprit to be the implicit dependency of buffer used at the line above.

Since there's no reference to this module in this repo I presume that it's accessible through the browserify module? Considering it's a key piece for this module to work, why not define the dependency in this package?

That said, there's a fix but it's not yet released (v3.4.2), so my temporary workaround to this was to override the implicit dependency, see: https://github.com/CartoDB/cartodb/pull/5171/files The fix was released in buffer v3.4.2

cc @tanem who seem to have suffered similar issues, perhaps this also works for you

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Examinez source-map-support.js à la ligne 111 ainsi que les métadonnées du paquet de ce dépôt ; vérifiez comment browserify fournit buffer. Le travail est terminé lorsque la gestion des dépendances est explicite et que le comportement pertinent d’installation ou de packaging a été vérifié par rapport au scénario browserify signalé.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript
Domaine
devtools
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.