Add find_symlink_node() to allow l* methods (lstat, lchown, etc) to use find_node like functionality about filer HOT 4 OPEN

Can you give an example of why you would want to do this?

from filer.

Created this issue on behalf of @humphd, but I'm currently trying to implement lchown(). Though I'm trying to get the metadata for a symbolic link, currently the find_node() method will try to de-reference and get the metadata for a regular link.

from filer.

My preference is to add a new function find_symlink_node instead of adding logic to find_node. Adding new functionality to find_node will make it more difficult to comprehend (it's already difficult) and more difficult to reason about when debugging. In this case I think two comprehensible functions is better than one function that tries to be too clever. You also have the benefit of making your improvements without disturbing other code that relies on find_node.

Have a look at the code for lstat; It also operates on symlinks. You can probably lift that logic out into file_symlink_node and rewrite lstat to use it as well.

from filer.

Good idea. Morphed this to be about adding a new function for the non-dereferencing case.

from filer.