-
Notifications
You must be signed in to change notification settings - Fork 793
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
Nextcloud client 3.1 syncs "ignored files" #2728
Comments
@camilasan could be related to your work in that area? rings any bell? |
Not only applies to per-instance ignore lists but also for global ignore list (in Settings→General). |
this slows down my client extremely. How can I go back to 3.0.3 under ubuntu? There's no 3.0.3. package in the repository anymore. |
You might find this link helpful, https://download.nextcloud.com/desktop/releases/ |
Thanks a lot. It works for the moment. |
Hmm so the global ignore file seems to work for me and it also appears in the log:
I'm just guessing, but @camilasan is it possible that the new path building in the directory traversal function has a bug? |
@libeanim I also have this issue. |
I noticed this with 3.1.0 and 3.1.1 on Windows as well. |
This isn't just a Windows problem. I noticed the same thing when I upgraded Nextcloud using the Linux AppImage yesterday. Unfortunately, I'm not sure what version I came from, but upgrading to 3.1.1 caused it to try to sync all of my excluded files. I downgraded to 3.0.3 as mentioned here and I still have the same problem. EDIT: The following workaround/hackery only works for me on 3.0.3, but not 3.1.1 However I noticed that if I move my excluded files out of the sync path and let the folders sync sans the ignored files it syncs fine (of course, since it doesn't have to consider them). Then, if I move the ignored files back into the folder once the initial sync finishes successfully the files are properly ignored -- at least in my very small test case. In one of my problem folders I had two subfolders, one with 50 or so fairly large source archives within a couple dozen folders. When I moved that folder into place it had no problems and did not synchronize them as the whole folder was on the ignore list. However, I had an issue with moving my second folder into place which had 15,357 folders and 330,359 files -- basically it at up all 119GB of my RAM before I finally had to kill it. But, that's a separate issue I think. So, there is some disparity between the ignore list when it comes to a full sync (or a force resync, presumably) and an update sync (when it detects a file change). |
I have this issue with 3.1.1 on windows. Folder specific ignore list doesn't work, global one does. |
Thanks @qu1ck! I was getting really frustrated by not being able to sync a couple of folders due to this bug. Adding the exception globally did fix the problem, albeit not a perfect solution. So, it seems that the client it's finding/reading/parsing the folder-specific sync excludes. At least that gets us closer to the right direction. I've been investigating this one myself as well, but I'm still trying to get the "lay of the land" as it were with the Nextcloud Client source. |
If I may provide some small insight into my actions dealing with this, I had to exclude log folders & contents, adding just Logs/ did not work. However, adding both 'Logs/' and 'Logs/*' made the entire folder plus contents be ignored as it should. Not sure if it is useful information, but that worked well enough for me. |
I will investigate it. |
Have the same problem / issue too. Upgraded from 3.0.3 auf 3.1.1. Now the the client would sync 300 GB off Data which are excluded. Downgrade to 3.0.3 the ignore list works. Problem solved with an older client. |
Curious, have you tried the method I spoke of a bit above here? it worked out for me in that way, it may also work for you without the need to go back versions :) |
@libeanim I have been having the same issue, but after the following fix worked, I think I know the bug's source.
After trying both, my sync excludes are working and I think I know why. Let's walk through our process again. Pretend we want to exclude the directory
However, if instead of creating foo, we copy foo into the synced folder, nextcloud ignores it (as it should). What's the difference? Theory OS: Windows 7 | NC Version 3.1.1 |
I found the opposite of #2728 (comment) Global Ignore list didn't work but folder ignore list did. Standard format Its like it wasn't reading the list at all, or maybe the working dir is incorrect from root ? (windows 3.1.2) |
Are you sure that this is a related problem? I see the same 2 problems when I update from 3.0.3 to 3.1.x:
I'm not sure if these two are related. But I hope, that this is the case, so solving one problem would also solve the other ;-) |
I don't know exactly what the problem is but I'll stick now with 3.0.3 for a while. |
Staying with version 3.0.3 is the only option for me at the moment. However, with version 3.0.3 I have this additional problem: #324 So no working client for me right now... :-/ |
This should be simpler, in the GUI to have an option to deselect what you don't want to synchronize, is that so complex to do? |
Previously the .sync-exclude.lst file of the sync root directory was not added to the exclude files, because the current logic did only recognize .sync-exclude.lst files when their containing directory was discovered during the discovery phase. Therefore the sync root .sync-exclude.lst file was never discovered. See also ExcludedFiles::traversalPatternMatch(). Fix: #3830, #2728 Signed-off-by: Felix Weilbach <[email protected]>
Previously the .sync-exclude.lst file of the sync root directory was not added to the exclude files, because the current logic did only recognize .sync-exclude.lst files when their containing directory was discovered during the discovery phase. Therefore the sync root .sync-exclude.lst file was never discovered. See also ExcludedFiles::traversalPatternMatch(). Fix: #3830, #2728 Signed-off-by: Felix Weilbach <[email protected]>
Previously the .sync-exclude.lst file of the sync root directory was not added to the exclude files, because the current logic did only recognize .sync-exclude.lst files when their containing directory was discovered during the discovery phase. Therefore the sync root .sync-exclude.lst file was never discovered. See also ExcludedFiles::traversalPatternMatch(). Fix: #3830, #2728 Signed-off-by: Felix Weilbach <[email protected]>
@camilasan "should" or "is" fixed? :) |
I was too eager to close the issue :) |
I would like to test both of these PRs, but there are no macos artifacts generated, and I'm having trouble compiling from source due to homebrew qt. If anyone can confirm that the root |
@dasaweb I just grabbed the latest nightly, and it looks like things are working! Huzzah! |
Same problem, ignore list not working. |
@bretthl Release 3.4.0 RC1 came out today (https://github.com/nextcloud/desktop/releases/tag/v3.4.0-rc1) that fixes #3882 and #2723. This should help. You can install this RC and help testing it ;-) |
Thanks for that, I will start using it and comment. |
3.4.1 RC1 is working with my testing using the ignore pattern somedir/* to suspend sync of all files in somedir. Keep up the good work, this app is awesome. |
Thank you for the pre-release. As expected, it works on MacOS as well with all the patterns in root and sub-directory being inherited correctly. Creating a Although, is there any |
isn't the correct pattern just somedir/ without the asterisk? won't somedir/* scan all the files in the subfolder anyway without syncing them afterwards instead of just leaving out the whole folder for scan? |
No
As far as my understand goes, Closing this issue as it seems that the original issue is resolved. Thanks for your help! |
Your are correct. I've been playing with both options and probably best to leave out the asterisk. Can anyone point me to a list of "flags" with explanations of each? Been awhile since I have used these. |
I was now brave enough to test the 3.4.0 RC1 on Win11, and tataaaaaa: It works! Finally having an up-to-date Nextcloud client again after a year is great. THANK YOU to everyone who contributed here! 💖 |
Previously the .sync-exclude.lst file of the sync root directory was not added to the exclude files, because the current logic did only recognize .sync-exclude.lst files when their containing directory was discovered during the discovery phase. Therefore the sync root .sync-exclude.lst file was never discovered. See also ExcludedFiles::traversalPatternMatch(). Fix: #3830, #2728 Signed-off-by: Felix Weilbach <[email protected]>
I still have this issue on Version 3.4.2 (Ubuntu).
set in both the "general settings" and the "..." submenu are leading to the ".git" folder nor its content beeing ignored by nextcloud. Is that only on my side or do others have the same issue? |
Yeah me too |
Me as well, I'm on EndeavourOS, my client version is:
Steps to reproduce are:
I carefully read through all this thread and related resources to ensure I'm not just messing up my exclusion "pattern", but at this point I'm pretty sure I don't and this has to be broken again :( |
I have some new info to share. I just got this working on Windows by using two ignores. For example to ignore everything in a folder named
|
My observation is that |
Any further updates to this? I have a .sync-exclude.lst file in a subdirectory for excluding (as others are doing .git). If I have it in the global it exlcudes as I desire. But if the excludes are set in a .sync-exclude.lst where the .git directory is a subdirectory. Then I can see the sync log full of objects when I make a commit. So for reference.
Version 3.4.0 Windows. |
You're commenting on an Issue that has been closed out for several years. If experiencing something (even similar) please open a new dedicated Issue as it may have a different underlying cause. |
How to use GitHub
Expected behaviour
Nextcloud client should ignore files and folders that have been added to the "Ignored Files" list through the UI.
Actual behaviour
Nextcloud client syncs folders that are in the "ignored files" list since the update to 3.1.
I've experienced this bug on "folder sync-connection"-level (see "Steps to reproduce" if unclear), it is also possible that it happens when using the global ignore file.
Steps to reproduce
Client configuration
Client version: 3.1.0-20201214.180315.7f4ffd0f7-1.0~focal1
Using Qt 5.12.8, built against Qt 5.12.8
Using Qt platform plugin 'xcb'
Using 'OpenSSL 1.1.1f 31 Mar 2020'
Running on Ubuntu 20.04.1 LTS, x86_64
Installation path of client: /usr/bin/nextcloud
Logs
Haven't seen any errors in the logs, it seems like a general client problem. If it is only my installation I can add this information.
The text was updated successfully, but these errors were encountered: