thecodrr / fdir

⚡ The fastest directory crawler & globbing library for NodeJS. Crawls 1m files in < 1s

Home Page:https://thecodrr.github.io/fdir/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Symlinks are ignored

aaronjensen opened this issue · comments

Ideally, symlinks would be followable or this would be an option. At the moment, they are completely ignored:

https://github.com/thecodrr/fdir/blob/master/src/api/shared.js#L48-L54

This appears to be causing this downstream issue in snowpack: FredKSchott/snowpack#2969

I am working on symlink support however it will affect performance greatly. It will be out and about soon though as an option withSymlinks probably.

That's great, thank you.

Just found this lib and I'm really interested in symlink support too! Any ETA on when can we expect it?

@DominikLevitsky I can't give an ETA but it is in the works. The main issue is implementing it safely and in a performant way. I will be posting updates here so hang on tight, it will be out in the next version for sure.

I am working on symlink support however it will affect performance greatly. It will be out and about soon though as an option withSymlinks probably.

Would it still "affect performance greatly" if the root directory is a symlink? ...as opposed to starting a real directory but possible symlinks within.

@peterbe no, the performance penalty is only for symlink resolution. Afterwards, fdir will operate normally. I have a basic PoC ready which i'll be pushing for testing soon.

Symlinks support is almost done. Just have to benchmark the performance impact when withSymlinks is not there.

Thanks!