search

raszi / node-tmp

Temporary file and directory creator for node.js
MIT License
736 stars 92 forks source link

Get rid of the file descriptor, close the file on creation and hand out only the path #241

Closed silkentrance closed 10 months ago

silkentrance commented 4 years ago

The file descriptor and the file that was opened by tmp has had some issues in the past, leading to a rather complex code base, including options such as discardDescriptor and detachDescriptor.

Getting rid of the file descriptor altogether seems to be a good solution that will eliminate a lot of rather complex code and which will also add to the overall stability of tmp.

This will be a breaking change that should be introduced with v0.3.0.

rijnhard commented 3 years ago

related https://github.com/raszi/node-tmp/issues/267

raszi commented 3 years ago

Good points and I agree to the proposal.

silkentrance commented 10 months ago

closing since the original maintainer shows no interest in this project anymore.