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

emit aboutToRemoveAllFiles causes owncloudcmd to hang wo/ prompt #8804

Closed
SamuAlfageme opened this issue Jul 5, 2021 · 3 comments
Closed

Comments

@SamuAlfageme
Copy link
Contributor

I was just hit by this after discovery:

07-05 12:24:44:910 [ info sync.networkjob.lscol ]:	LSCOL of QUrl("https://<acct>@<server>/remote.php/webdav/<path>/abc") FINISHED WITH STATUS "OK"
07-05 12:24:44:911 [ info sync.discovery ]:	Processing "abc/abc" | valid: true/true/false | mtime: 1625479485/1625479485/0 | size: 0/0/0 | etag: "28b7e:1625479485.077"//"" | checksum: ""//"" | perm: 0x8b1058//0x7f1cb9635998 | fileid: "ZW9zaG9tZS1pMDE6MTY2Nzgy"//"" | inode: 3301694746119/3301694746119/ | type: CSyncEnums::ItemTypeDirectory/CSyncEnums::ItemTypeDirectory/CSyncEnums::ItemTypeFile
07-05 12:24:44:911 [ info sync.discovery ]:	Discovered "abc/abc" CSyncEnums::CSYNC_INSTRUCTION_REMOVE OCC::SyncFileItem::Down CSyncEnums::ItemTypeDirectory
07-05 12:24:44:914 [ info sync.discovery ]:	STARTING "abc/abc" OCC::ProcessDirectoryJob::ParentDontExist "abc/abc" OCC::ProcessDirectoryJob::NormalQuery
07-05 12:24:44:922 [ info sync.engine ]:	#### Discovery end ####################################################  150 ms
07-05 12:24:44:924 [ info sync.engine ]:	All the files are going to be changed, asking the user

emit aboutToRemoveAllFiles(side >= 0 ? SyncFileItem::Down : SyncFileItem::Up, callback);

... the command line client indefinitely hangs when this condition is met.

@michaelstingl
Copy link
Contributor

michaelstingl commented Jul 5, 2021

@SamuAlfageme run with --non-interactive as a workaround?
(https://doc.owncloud.com/desktop/advanced_usage/command_line_client.html)

@TheOneRing
Copy link
Member

Its not handled

@TheOneRing
Copy link
Member

Fixed in #8805

TheOneRing added a commit that referenced this issue Jul 6, 2021
TheOneRing added a commit that referenced this issue Jul 6, 2021
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