Skip to content

Edit file instantly incorrectly re-uploads and overwrites file on server #14251

Description

@lostminds

Describe the bug
After a recent update pressing the Edit button in the file browser now not only downloads the file and opens it in my local text editor (BBEdit) as previously, but also then instantly re-uploads the file overwriting the file on server with a new copy of the file. As far as I can tell the content seems to be the same, but it gives me a scare and means files get changed on the server and a new timestamp even if they haven't been edited.

To Reproduce
Steps to reproduce the behavior:

  1. Select a text file on a remote server
  2. Click on Edit to download the file and open it in your local text editor
  3. See the file open (do nothing)
  4. See Cyberduck immediately upload the file again, overwriting the file on the server with a new copy

Expected behavior
Expect the file to open in the local editor and not be uploaded to the server until the file has been changed and saved.

Desktop (please complete the following information):

  • OS: macOS 13.2
  • Version Version 8.5.5 (39213)

Additional context
I've had no problems with this functionality for many years using BBEdit and Cyberduck together. So something must have changed in either app (or macOS) recently making Cyberduck think the file it has just downloaded is changed and needs to be re-uploaded immediately.

This is very troublesome since I regularly use the Edit function to simply look at web source files in my editor, with no intent to edit them. And now with this bug I don't dare use the feature as it will immediately change the file on the server in unexpected ways.

Activity

  1. nicoh88 commented on Feb 20, 2023

    @nicoh88

    I can confirm. I have the same problem. It's really annoying with the new file timestamp.

  2. added this to the 8.5.7 milestone on Feb 21, 2023
  3. self-assigned this
    on Feb 21, 2023
  4. dkocher commented on Mar 2, 2023

    @dkocher
    Contributor

    There is a checksum comparison that should skip any upload when the file open in the editor and we receive a filesystem event but the file has not changed

    // If checksum still the same no need for save
    final Checksum current;
    try {
    progress.message(MessageFormat.format(
    LocaleFactory.localizedString("Compute MD5 hash of {0}", "Status"), temporary.getName()));
    current = ChecksumComputeFactory.get(HashAlgorithm.md5).compute(temporary.getInputStream(), new TransferStatus());
    }
    catch(BackgroundException e) {
    log.warn(String.format("Error computing checksum for %s. %s", temporary, e));
    return Worker.empty();
    }
    if(current.equals(checksum)) {

  5. dkocher commented on Mar 3, 2023

    @dkocher
    Contributor

    Some cleanup in #14307.

  6. added a commit that references this issue on Mar 7, 2023
  7. added theissue type on Sep 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions