M2Team / NanaZip

The 7-Zip derivative intended for the modern Windows experience
https://sourceforge.net/projects/nanazip/
Other
8.92k stars 224 forks source link

Nanazip Does Not Handle Permissions Correctly - "Access is denied" Instead of Elevation Prompt #509

Open MNLierman opened 1 week ago

MNLierman commented 1 week ago

Expected Behavior: When attempting to complete an action that required elevated permissions such as creating a zip file from a folder in the root of the C drive, or other areas that require admin rights, I would expect that Nanazip would, 1) Test the permissions through various APIs available to do so, and if Administrative rights are required to save a zip file in that location, 2) Nanazip should inform the user that the directory requires an elevation request, and, 3) In this dialog give the user the option to save somewhere else if they do not wish to elevate permissions or they are unable to do so.

Actual Behavior: The actual behavior is frustrating, Nanazip displays a generic "Access is denied. cannot open file {path}" which looses the configuration the user created for the zip file creation and it also does not offer an alternative save location. I'm actually surprised that when the "Access is denied" message was created, that permissions checks and elevation were not added. At the very least, Nanazip should be offering to save the zip somewhere else if it encounters a permissions issue.

Steps to Reproduce:

  1. Select any files or folders in the root of the C drive, right click, Nanazip, and attempt to create a archive.
  2. Choose your settings and press OK.
  3. An "Access is denied." message is encountered instead of a choice dialog to handle the permissions appropriately.

image