felixge / felixge/node-sandboxed-module

Circular requires that work with regular requires do not work with sandboxed-module

Offen
#50 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
343
Forks
41
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

[Node.js supports circular requires](https://nodejs.org/api/modules.html#modules_cycles)
The behavior can be odd or unexpected, but it is supported.

In some cases though, this library is loading these circular requires differently, causing them to fail.

I posted an example of such a situation here: https://github.com/thoughtless/circular_require_demo

Another example is the [NPM module](https://www.npmjs.com/package/aws-sdk) `aws-sdk`. It has a circular require between [service.js](https://github.com/aws/aws-sdk-js/blob/08de9516754ea98b88bcc186c8b6ffbf89458502/lib/service.js#L1) and [core.js](https://github.com/aws/aws-sdk-js/blob/08de9516754ea98b88bcc186c8b6ffbf89458502/lib/core.js#L73).
`require("sandboxed-module").require("aws-sdk")` will fail.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.