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

Recursive file permission changes not applying recursively in Cyberduck 8.5.2 (38799) for Windows #14004

Closed
elohanlon opened this issue Dec 16, 2022 · 3 comments · Fixed by #14611
Assignees
Labels
bug interface User Interface
Milestone

Comments

@elohanlon
Copy link

elohanlon commented Dec 16, 2022

Describe the bug
In the latest Windows version of Cyberduck – version 8.5.2 (38799) – when I select a remote server directory and try to recursively change file permissions for that directory, only the top directory's permissions are changed. None of the child files or subdirectories are modified.

By comparison, trying exactly the same operation in the latest Mac version – also Version 8.5.2 (38799) – works properly. I tried this Mac test on the same remote server directory.

To Reproduce
Steps to reproduce the behavior:

  1. Install a fresh copy of Cyberduck for Windows and keep all default settings.
  2. Connect to a server over SFTP with username and password (or username and ssh key).
  3. Create a directory on the server (let's call it "permission-test-dir") with some nested files and subdirectories, making sure that you're creating this directory in a place where you have the necessary permissions to create new files.
  4. Right click on "permission-test-dir" and select "Info".
  5. In the dialog that opens, select the "Permissions" tab.
  6. Type in the new, desired permission value (I've done tests with 770, 771, 776, and other variations).
  7. Click "Apply changes recursively".
  8. Click "Continue" on the prompt that follows. (I am only presented with this prompt once.)
  9. Right click on "permission-test-dir" and select "Refresh" (to ensure that we're looking at the latest permission state).
  10. Observe that "permission-test-dir" permissions have changed to the new value (good).
  11. Observe that permissions for files and subdirectories inside "permission-test-dir" have NOT been recursively changed to the desired value (bad).

I also wanted to note here that in step 8, when trying the same test in the Mac version, I am prompted with more than one "Continue" prompt – one for the directory I am trying to modify, and an additional one for each of the subdirectories.

Expected behavior
In the "Steps to reproduce" above, I would have expected that in step 11, all child files and subdirectories inside "permission-test-dir" would have been updated to have the same permissions as "permission-test-dir".

Desktop (please complete the following information):

  • OS: Verified failure case on Windows 11 and Windows 10.
  • Cyberduck Version 8.5.2 (38799)

No problem in Cyberduck 8.5.2 (38799) on macOS 12.6.2.

Log Files
My Cyberduck log file was empty.

@dkocher dkocher added the ftp FTP Protocol Implementation label Mar 17, 2023
@tatze96 tatze96 added sftp SFTP Protocol Implementation and removed ftp FTP Protocol Implementation labels Apr 3, 2023
@AliveDevil AliveDevil self-assigned this May 3, 2023
@AliveDevil AliveDevil added bug interface User Interface and removed sftp SFTP Protocol Implementation labels May 3, 2023
@AliveDevil
Copy link
Contributor

Thanks for reporting!

Found the root cause of this. Will be fixed in an upcoming release.

@dkocher dkocher added this to the 8.6.0 milestone May 3, 2023
@AliveDevil
Copy link
Contributor

Regression in … 2016 😳
09e34db

AliveDevil added a commit that referenced this issue May 3, 2023
@elohanlon
Copy link
Author

@AliveDevil Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug interface User Interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants