fs: fix invalid stat when path is a symbolic link

kofemann / simple-nfs

dcache nfs server to export regular file system

Apache License 2.0

14 stars 14 forks source link

fs: fix invalid stat when path is a symbolic link #14

Closed Toilal closed 5 years ago

Toilal commented 5 years ago

This patch fix support for symbolic links on windows. (I actually look for an alternative to WinNFSD).

kofemann commented 5 years ago

@Toilal thanks for PR! Could you please add 'Signed-off-by: (git commit -s)

Toilal commented 5 years ago

It's already signed (I have forced-push to add it yesterday).

Toilal commented 5 years ago

Would you consider using 777/555 for file permissions when server is running on windows ? Problem is that I need to be root to access the mounted directory with actualy file permissions.

Toilal commented 5 years ago

I still try to perform npm install on a nodeJS project, it always fails with various IO error for now.

I'm pretty sure it's because of the remap method implementation. Any idea to enhance this ?