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

Unarchiving read-only folders fails #391

Open
omerzi opened this issue Dec 18, 2023 · 1 comment
Open

Unarchiving read-only folders fails #391

omerzi opened this issue Dec 18, 2023 · 1 comment

Comments

@omerzi
Copy link

omerzi commented Dec 18, 2023

What version of the package or command are you using?

v3.5.1

What are you trying to do?

Trying to extract a tar.gz file (which I suspect might replicate with other archive formats) that contains a read-only folder along with its files.

What steps did you take?

Create an archive with a write-protected folder:

  1. mkdir -p TestArchive/writeprotectedfolder
  2. touch TestArchive/writeprotectedfolder/file1.txt
  3. touch TestArchive/writeprotectedfolder/file2.txt
  4. chmod -w TestArchive/writeprotectedfolder/
  5. tar -xzf example.tar.gz
  6. Untar using the Unarchive function

What did you expect to happen, and what actually happened instead?

An error returned:
reading file in tar archive: /path/TestArchive/writeprotectedfolder/file2.txt: creating new file: open /path/TestArchive/writeprotectedfolder/file2.txt: permission denied

How do you think this should be fixed?

The issue arises when the code creates the writeprotectedfolder with its original read-only permissions (https://github.com/mholt/archiver/blob/v3.5.1/tar.go#L264). Consequently, we encounter an inability to write file1.txt and file2.txt into it. I intended to initiate a pull request to address this, but I am unable to do so as the master branch points to v4, and there isn't a v3 branch available.

Please link to any related issues, pull requests, and/or discussion

I believe that this pull request has caused the issue: #133.

@omerzi omerzi changed the title Unarchive protected folders fails Unarchiving read-only folders fails Dec 18, 2023
@mholt
Copy link
Owner

mholt commented Dec 18, 2023

I'm not working on v3 anymore. But this is good to know for v4, in case we do add unarchiving capabilities: #311

Maybe a better solution would be to apply folder permissions after extraction is complete.

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

No branches or pull requests

2 participants