NativeScript / NativeScript/nativescript-cli
forced require of non-existant buffer
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 1.1k
- Forks
- 204
- Ø Merge
- 1 T. 9 Std.
- Gemergte PRs (30 T.)
- 8
Beschreibung
Environment
Provide version numbers for the following components (information can be retrieved by running tns info in your project folder or by inspecting the package.json of the project):
- CLI: 6.8.0
- Cross-platform modules: 6.5.15
- Android Runtime: 6.5.3
- iOS Runtime: 6.5.2
Describe the bug
After updating the node_modules in my project including js-base64, starting the app resulted in an exception on trying to require the module buffer, which was never anywhere explicitly required.
According to the author (https://github.com/dankogai/js-base64/issues/126#issuecomment-674838031) the new version does check for existence of the Buffer object to maybe use it if available.
Expected behavior
Either do not add the require('buffer') or add buffer to the dependencies if not already there.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, den App-Start mit den aufgeführten NativeScript-Versionen zu reproduzieren, und untersuche den Abhängigkeitspfad von js-base64 bis zum nicht verfügbaren buffer-Modul. Überprüfe die Abhängigkeitserklärungen der CLI und die Behandlung des Starts; abgeschlossen ist die Aufgabe, wenn die App nicht mehr versucht, ein nicht verfügbares require auszuführen, entweder weil die Abhängigkeit verfügbar ist oder das require nicht eingeführt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- android, ios, javascript
- Bereich
- cli, mobile
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100