Repository navigation
Deprecate fs.exists or fix its API #103
Description
Activity
Yes! Option 1 is my vote. This is a tiny black spot on the Node fs API.
+1 for deprecating. This was already being discussed in nodejs/node-v0.x-archive#8418.
fs.access()an alternative is also up for PR nodejs/node-v0.x-archive#8714. Since both of those are mine, I'd be glad to do the work here.+1 on option 1.
👍 for deprecating
Sounds reasonable. @cjihrig Go for it.
@cjihrig yes please! The fix looks pretty solid and so far everyone is for deprecating it.
Hello!
I am pleased to see your valuable contribution to this project. Would you
please mind answering a couple of questions to help me classify this submission
and/or gather required information for the core team members?Questions:
- Issue-only Does this issue happen in core, or in some user-space
module from npm or other source? Please ensure that the test case
that reproduces this problem is not using any external dependencies.
If the error is not reproducible with just core modules - it is most
likely not a io.js problem. Expected:yes - Which part of core do you think it might be related to?
One of:tls, crypto, buffer, http, https, assert, util, streams, smalloc, cluster, child_process, dgram, c++, docs, other(label) - Which versions of io.js do you think are affected by this?
One of:v0.10, v0.12, v1.0.0(label)
Please provide the answers in an ordered list like this:
- Answer for the first question
- Answer for the second question
- ...
Note that I am just a bot with a limited human-reply parsing abilities,
so please be very careful with numbers and don't skip the questions!In case of success I will say:
...summoning the core team devs!.In case of validation problem I will say:
Sorry, but something is not right here:.Truly yours,
Caine.Responsibilities
- indutny: crypto, tls, https, child_process, c++
- trevnorris: buffer, http, https, smalloc
- bnoordhuis: http, cluster, child_process, dgram
- Issue-only Does this issue happen in core, or in some user-space
- Yes
- fs
- v0.12
On Dec 8, 2014, at 15:30, Michael Caine notifications@github.com wrote:
Hello!
I am pleased to see your valuable contribution to this project. Would you
please mind answering a couple of questions to help me classify this submission
and/or gather required information for the core team members?Questions:
Issue-only Does this issue happen in core, or in some user-space module from npm or other source? Please ensure that the test case that reproduces this problem is not using any external dependencies. If the error is not reproducible with just core modules - it is most likely not a io.js problem. Expected: yes
Which part of core do you think it might be related to? One of: tls, crypto, buffer, http, https, assert, util, streams, smalloc, cluster, child_process, dgram, c++, docs, other (label)
Which versions of io.js do you think are affected by this? One of: v0.10, v0.12, v1.0.0 (label)
Please provide the answers in an ordered list like this:Answer for the first question
Answer for the second question
...
Note that I am just a bot with a limited human-reply parsing abilities,
so please be very careful with numbers and don't skip the questions!In case of success I will say: ...summoning the core team devs!.
In case of validation problem I will say: Sorry, but something is not right
here:.Truly yours,
Caine.Responsibilities
indutny: crypto, tls, https, child_process, c++
trevnorris: buffer, http, https, smalloc
bnoordhuis: http, cluster, child_process, dgram
—
Reply to this email directly or view it on GitHub.5 remaining items
- added a commit that references this issue
on Dec 19, 2014 Awesome news :) Thanks a ton.
- added a commit that references this issue
on Jan 12, 2015 I am sad at this deprecation.
We have an npm module now: https://github.com/sindresorhus/path-exists
- added a commit that references this issue
on Jun 25, 2015 /facepalm How was this not sufficient:
fs.exists() should not be used to check if a file exists before calling fs.open(). Doing so introduces a race condition since other processes may change the file's state between the two calls. Instead, user code should call fs.open() directly and handle the error raised if the file is non-existent.
Because I don't want to open the file.
Please see: #1592
- locked and limited conversation to collaborators
on Oct 10, 2015
fs.existsis infamous for having an inconsistent non-nodeback API that confuses new users often and can be a pain spot.I see two good alternatives:
fs.statinstead + discuss the inherent problem with usingexists(race condition). Add a big warning. Optionallyconsole.loga deprecation notice when the server is first started.Personally I'm for the first. Let's clean up
fs:)