Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unnecessary path structure in ZIP created with archive feature #6241

Closed
cyberduck opened this issue Sep 19, 2011 · 3 comments
Closed

Unnecessary path structure in ZIP created with archive feature #6241

cyberduck opened this issue Sep 19, 2011 · 3 comments
Assignees
Labels
enhancement fixed sftp SFTP Protocol Implementation
Milestone

Comments

@cyberduck
Copy link
Collaborator

bcbb62d created the issue

When zip (Create Archive) is used for directory - result is the directory containing all directories as path to original directory. It is also security risk, because user on other side can see structure of directories.

Example: directory_a ==> directory_1/directory_2/directory_3/directory_4/directory_a/file_1 ...

I expect the same behavior as in the Terminal:

zip -r foo.zip directory_a

result is directory_a containing file_1

@cyberduck
Copy link
Collaborator Author

@dkocher commented

This is because we always path absolute path names without changing the working directory. Unfortunately zip doesn't have a -C option like tar.

@cyberduck
Copy link
Collaborator Author

@dkocher commented

See also #6644.

@cyberduck
Copy link
Collaborator Author

@dkocher commented

~~ In ec5f2fd and 332a54d. ~~

In e870832.

@iterate-ch iterate-ch locked as resolved and limited conversation to collaborators Nov 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement fixed sftp SFTP Protocol Implementation
Projects
None yet
Development

No branches or pull requests

2 participants