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

Add unzip functionality to FileManager #22

Closed
psolom opened this issue Aug 6, 2016 · 9 comments
Closed

Add unzip functionality to FileManager #22

psolom opened this issue Aug 6, 2016 · 9 comments
Milestone

Comments

@psolom
Copy link
Owner

psolom commented Aug 6, 2016

Original request by @SlitYahoo and discussion is here

@fabriceci
Copy link
Contributor

fabriceci commented Aug 7, 2016

I thinks it's a good idea.

To handle the security issues for the unzip, a possible solution is the feature request I did.

@simogeo
Copy link
Contributor

simogeo commented Aug 12, 2016

@servocoder : what about unzipping files on /tmp/ repo (sys_get_temp_dir ()) and check files individually to remove unwanted before copying the folder into the desired location ?

@psolom
Copy link
Owner Author

psolom commented Aug 12, 2016

Splendid idea, we should try it definitely when it comes to implementation. Thanks.

@binaryhq
Copy link

binaryhq commented Apr 4, 2017

@servocoder Any update on this ?

@psolom
Copy link
Owner Author

psolom commented Apr 4, 2017

not yet, going to take care of this in the coming releases

@binaryhq
Copy link

binaryhq commented Apr 4, 2017

@servocoder thanks 👍 I'll be waiting

@psolom
Copy link
Owner Author

psolom commented Apr 7, 2017

I have just commited "unzip" branch that includes extract feature.
@Ningappa, @arvindkalbhor and other participants who are interested in this feature, please clone "unzip" branch, test it and give you feedback and bugs , if any. I'm going to releaze extract feature this weekend.

To extract zip archive you have to right click on zip file and select extract option at the bottom. Note, that capabilities option in JSON configuration file should include extract value. Extracted files will be restricted according to security setting in the config.php file (for PHP connector).

@binaryhq
Copy link

binaryhq commented Apr 7, 2017

Fantastic. It's working.

@psolom psolom added this to the 2.3.2 milestone Apr 8, 2017
@psolom
Copy link
Owner Author

psolom commented Apr 8, 2017

Released in 2.3.2

@psolom psolom closed this as completed Apr 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants