Skip to content

feat(zip): add support for symlink extraction#38

Closed
sephriot wants to merge 2 commits intomholt:mainfrom
spacelift-io:main
Closed

feat(zip): add support for symlink extraction#38
sephriot wants to merge 2 commits intomholt:mainfrom
spacelift-io:main

Conversation

@sephriot
Copy link
Contributor

@sephriot sephriot commented May 30, 2025

This PR fixes #37 issue

Signed-off-by: Jakub Wójtowicz <jakub.wojtowic@gmail.com>
@mholt
Copy link
Owner

mholt commented May 31, 2025

Thanks! I don't know much about this... is it typical for the symlink target to be stored as the content of the file in a zip archive?

@sephriot
Copy link
Contributor Author

sephriot commented Jun 1, 2025

Thanks! I don't know much about this... is it typical for the symlink target to be stored as the content of the file in a zip archive?

According to Wikipedia, that is exactly the case

A symbolic link contains a text string that is automatically interpreted and followed by the operating system as a path to another file or directory. This other file or directory is called the "target".

ZIP standard does not specify any special way to handle symlinks.
Symlink in a ZIP archive is just a "file" with symbolic link mode flag set, containing symlink target.

Signed-off-by: Jakub Wójtowicz <jakub.wojtowic@gmail.com>
@sephriot
Copy link
Contributor Author

sephriot commented Jun 2, 2025

Moved to: #39 due to new commits in forked main branch

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.

Symlinks in ZIP have blank LinkTarget

2 participants