Skip to content

Check files symlink support#398

Open
inqrphl wants to merge 2 commits into
mainfrom
check-files-symlink-support
Open

Check files symlink support#398
inqrphl wants to merge 2 commits into
mainfrom
check-files-symlink-support

Conversation

@inqrphl

@inqrphl inqrphl commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

default golang filepath.Walk did not traverse into the recursive directories. need to check if a filesystem item is a link using fs.readLink

add a new file walker, it can resolve file links, and keeps track of visited directories to prevent infinite recursion.

add new argument followSymlinks and new attribute is_symlink .

followSymlinks is true by default

is_symlink is true only if the filesystem item is a link

The code also tracks if the filesystem item is reached using a link or not, independent of it being a link itself. This is not saved to an attribute.

Ahmet Oeztuerk added 2 commits June 30, 2026 10:09
add new argument followSymlinks and new attribute is_symlink

add a new file walker, it can resolve file links, and keeps track of visited directories to prevent infinite recursion
Add default value to the followSymlinks argument definition

Change is_symlink behavior: it is only toggled on if the file/directory itself is a symlink

change the unit of is_symlink to UBool, its filters can now parse bool values
@lgmu

lgmu commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

symbolic links for linux and windows work, but junctions are not recognized yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants