-
-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
realpathSync to realpathSync.native #5
Comments
Sounds like this should be fixed in Node.js instead. |
As I understand it, The promise version |
Thanks @Richienb I recommend testing in Windows too moving forward as this issue was surfaced as a result of Window's specific behavior. |
In Windows, doing
fs.realpathSync(os.tmpdir())
doesn't resolve the symlink. You must dofs.realpathSync.native(os.tmpdir())
.However, I'm not sure if it is always safe to use: nodejs/node#39721
The text was updated successfully, but these errors were encountered: