fshost / node-dir

Recursive asynchronous file and directory operations for Node.js
MIT License
221 stars 44 forks source link

Potential "stack size exceeded" #9

Closed reverofevil closed 9 years ago

reverofevil commented 10 years ago

The default stack depth in Node.js is 492 while max file path in almost every filesystem is unlimited (http://en.wikipedia.org/wiki/Comparison_of_file_systems). Use explicit stack, please.

fshost commented 9 years ago

If needed you can change the default stack depth for node. It is implicitly assumed that the use cases for this are within node parameters, and if not, perhaps another library would be a better choice. However, if you are having an actual error (as opposed to theoretical) please provide the code and use case (or contribute a pull request) and I might be willing to revisit this. (If I have failed to understand your concern, perhaps you could be more explicit.)