On Windows, patch Node's lstat to work around permissions error. Fixes #1101 #596

New Issue

fergalmoran · 2025-08-09T17:16:54Z

fergalmoran commented

2025-08-09 17:16:54 +00:00

Originally created by @SteveSandersonMS on 7/11/2017

It's not great to have to patch Node internals like this, but it's the best we've got in the short term. See #1101 for details of the problem this solves.

This logic was tested on Node versions 0.12.0, 4.6.2, 6.11.0, and 8.1.3 (Windows 10, 64 bit).

Longer term, we need to work with the Node folks to get a proper fix into Node itself and then make this patching logic conditional on older versions of Node.

*Originally created by @SteveSandersonMS on 7/11/2017* It's not great to have to patch Node internals like this, but it's the best we've got in the short term. See #1101 for details of the problem this solves. This logic was tested on Node versions 0.12.0, 4.6.2, 6.11.0, and 8.1.3 (Windows 10, 64 bit). Longer term, we need to work with the Node folks to get a proper fix into Node itself and then make this patching logic conditional on older versions of Node.

fergalmoran closed this issue

2025-08-09 17:16:54 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/JavaScriptServices#596