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

Comments

@cyberduck
Copy link
Collaborator

@cyberduck cyberduck commented Sep 19, 2011

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

@cyberduck cyberduck commented Sep 20, 2011

@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.

Loading

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Dec 19, 2012

@dkocher commented

See also #6644.

Loading

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Jul 9, 2013

@dkocher commented

~~ In ec5f2fd and 332a54d. ~~

In e870832.

Loading

@cyberduck cyberduck closed this Jul 9, 2013
@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.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants