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

[linux] Virtual files inside a folder are not downloaded after disabling VFS support #11240

Closed
saw-jan opened this issue Sep 28, 2023 · 16 comments

Comments

@saw-jan
Copy link
Member

saw-jan commented Sep 28, 2023

Describe the bug

After disabling VFS support files inside the folder are not downloaded and .owncloud files of downloaded files are also not removed.

Expected behavior

All files are downloaded and synced to the sync folder

Steps to reproduce the issue

  1. Add some files/folders-with-files to the server
  2. Add the account to the desktop client with VFS enabled
  3. Disable the VFS support
  4. Files on root level gets downloaded but respective .owncloued files are still there ❌
  5. Files inside some folders don't get downloaded ❌

Screenshots

Screenshot from 2023-09-28 11-56-48 Screenshot from 2023-09-28 11-58-01
VFS ON VFS OFF - root level files are downloaded but .owncloud files still remain ❌
Screenshot from 2023-09-28 11-58-07
Files inside a folder are not downloaded ❌

Client version number

owncloudcmd ownCloud 5.0.0.12187-rc.2
https://github.com/owncloud/client/commit/5a9119bdfd0794421f9ac7888f05b3ab16df4e59
Libraries Qt 6.4.3, OpenSSL 3.1.2 1 Aug 2023
Using virtual files plugin: suffix
OS: ubuntu-6.2.0-33-generic

Desktop environment (Linux only)

Ubuntu (gnome with nautilus)

Client log

ownCloud.log

@saw-jan saw-jan changed the title [linux] Virtual files are not downloaded after disabling VFS support [linux] Virtual files inside a folder are not downloaded after disabling VFS support Sep 28, 2023
@TheOneRing TheOneRing added this to the Desktop 5.1 milestone Sep 28, 2023
@TheOneRing
Copy link
Contributor

Are the two files actual placeholders, empty 1 byte files, or where they modified with a text editor as the client did not download the actual file when double clicking them.

@saw-jan
Copy link
Member Author

saw-jan commented Sep 28, 2023

Are the two files actual placeholders, empty 1 byte files, or where they modified with a text editor as the client did not download the actual file when double clicking them.

those files were synced from server to the client with vfs enabled and after disabling vfs they should be converted to actual file, right?

@TheOneRing
Copy link
Contributor

Only if they are not modified, if placeholder file is modified its detected as some random file with a stupid file name and ignored.
Your logs would tell you.

@saw-jan
Copy link
Member Author

saw-jan commented Sep 28, 2023

Only if they are not modified, if placeholder file is modified its detected as some random file with a stupid file name and ignored. Your logs would tell you.

I just synced with vfs on and then disabled vfs. so nothing touched in the sync folder.
I will report the logs as well.

@saw-jan
Copy link
Member Author

saw-jan commented Sep 28, 2023

I have attached the log in the description. Maybe something can be found there

@TheOneRing
Copy link
Contributor

I can see the file "short.txt.owncloud" in the logs with the same behaviour as INSTALL.sh.owncloud.
And in owncloud/docs-client-desktop#519 I can see a screenshot that that file was opened in a text editor.

@saw-jan
Copy link
Member Author

saw-jan commented Sep 28, 2023

And in owncloud/docs-client-desktop#519 I can see a screenshot that that file was opened in a text editor.

yeah, for owncloud/docs-client-desktop#519, only short.txt.owncloud is opened in the editor. But for this, I cleaned up everything and tried it fresh

@TheOneRing
Copy link
Contributor

Did the files still exist on the server, if you manage to have .owncloud files on your system that don't have an actual file on the server we would see something like this too.

@saw-jan
Copy link
Member Author

saw-jan commented Sep 28, 2023

The server has all the files. nothing got deleted

@TheOneRing
Copy link
Contributor

23-09-28 14:51:11:369 [ info sync.discovery ]:	Processing "f1/INSTALL.sh.owncloud" | valid: true/true/db | mtime: 1695645471/1695645471/0 | size: 499/1/0 | etag: "73c7dbc7acff9d85175d3bb3dcfd189e"//"" | checksum: "SHA1:c923cdd0b22dace6b0342b45134186fc45ddb3c1"//"" | perm: "WDNVR"//"" | fileid: "c80f2074-a213-43fd-a607-1a2ab1112e3a$27491a5b-e357-452f-bf4b-d03b51b22eae!eb168cc6-ea33-4997-9fab-82c1b27d1b7b"//"" | inode: 1053049/1053049/ | type: CSyncEnums::ItemTypeVirtualFile/CSyncEnums::ItemTypeVirtualFile/CSyncEnums::ItemTypeFile
23-09-28 14:51:11:369 [ info sync.discovery ]:	Processing "f1/example.sh.owncloud" | valid: true/true/db | mtime: 1695645443/1695645443/0 | size: 729/1/0 | etag: "9da29dc55e4c5851709a2886b1de3c94"//"" | checksum: "SHA1:8645746e32eda723bc16d0dcbac4117040101f72"//"" | perm: "WDNVR"//"" | fileid: "c80f2074-a213-43fd-a607-1a2ab1112e3a$27491a5b-e357-452f-bf4b-d03b51b22eae!2bd6da9f-9e12-40ed-a960-9efb06294799"//"" | inode: 1053065/1053065/ | type: CSyncEnums::ItemTypeVirtualFile/CSyncEnums::ItemTypeVirtualFile/CSyncEnums::ItemTypeFile

For bot files it says it has no mtime on the server and no file size, it looks more like you copied a example.sh.owncloud file to the sync root.

@saw-jan
Copy link
Member Author

saw-jan commented Sep 28, 2023

hmm, that's not what I did though. Let me do it again in some other VMs

@saw-jan
Copy link
Member Author

saw-jan commented Sep 28, 2023

hmm, that's not what I did though. Let me do it again in some other VMs

Still same with Debian12. But also now noticed that re-enabling VFS deletes all the .owncloud files (but files are fine in the server)

@TheOneRing TheOneRing modified the milestones: Desktop 5.1, Desktop 6.0 Oct 10, 2023
@saw-jan
Copy link
Member Author

saw-jan commented Nov 8, 2023

Works for Windows - all vfs files are downloaded

@TheOneRing
Copy link
Contributor

This is a linux issue, what happens on linux?

@saw-jan
Copy link
Member Author

saw-jan commented Nov 9, 2023

Tested with ownCloud 5.1.1.12577-rc.1 a98b47 on Ubuntu 22.04
All files are downloaded after disabling the VFS ✔️
And no .owncloud files are present ✔️

@TheOneRing
Copy link
Contributor

great, if I just had understood the impact of your finding here earlier 🙈

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

3 participants