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

Resolve client 'hang' when attempting to sync a Unix pipe file #985

Merged
merged 3 commits into from
Jul 7, 2020

Conversation

abraunegg
Copy link
Owner

  • Resolve client 'hang' when attempting to sync a Unix pipe file

* Resolve client 'hang' when attempting to sync a unix pipe file
@abraunegg
Copy link
Owner Author

Client filesystem:

total 8
drwxrwxr-x.  2 alex alex  121 Jul  7 06:04 .
drwx------. 31 alex alex 4096 Jul  7 05:39 ..
-rw-rw-r--.  1 alex alex   25 Jul  7 05:39 dummy_file.txt
prw-rw-r--.  1 alex alex    0 Jul  7 05:55 my_pipe
lrwxrwxrwx.  1 alex alex   14 Jul  7 06:04 qwer.txt -> dummy_file.txt
lrwxrwxrwx.  1 alex alex   32 Jul  7 05:39 random_files -> /home/alex/OneDrive/random_files
lrwxrwxrwx.  1 alex alex   33 Jul  7 05:39 random_images -> /home/alex/OneDrive/random_images
lrwxrwxrwx.  1 alex alex   33 Jul  7 05:39 random_videos -> /home/alex/OneDrive/random_videos

Without patch:

Using 'user' Config Dir: /home/alex/.config/onedrive-personal/
Using 'system' Config Dir: 
Configuration file successfully loaded
Initializing the OneDrive API ...
Configuring Global Azure AD Endpoints
Opening the item database ...
All operations will be performed in: /home/alex/OneDrivePersonal
Application version: v2.4.3-6-ge23e063
Account Type: personal
Default Drive ID: 66d53be8a5056eca
Default Root ID: 66D53BE8A5056ECA!101
Remaining Free Space: 5368709070
Fetching details for OneDrive Root
OneDrive Root exists in the database
Initializing the Synchronization Engine ...
Syncing changes from OneDrive ...
Applying changes of Path ID: 66D53BE8A5056ECA!101
Uploading differences of ~/OneDrivePersonal
Processing root
The directory has not changed
Processing random_files
The directory has not changed
Processing random_images
The directory has not changed
Processing random_videos
The directory has not changed
Processing dummy_file.txt
The file has not changed
Processing qwer.txt
The file has not changed
Uploading new items of ~/OneDrivePersonal

[process hangs / infinite wait]

^C

With Patch:

Using 'user' Config Dir: /home/alex/.config/onedrive-personal/
Using 'system' Config Dir: 
Configuration file successfully loaded
Initializing the OneDrive API ...
Configuring Global Azure AD Endpoints
Opening the item database ...
All operations will be performed in: /home/alex/OneDrivePersonal
Application version: v2.4.3-6-ge23e063
Account Type: personal
Default Drive ID: 66d53be8a5056eca
Default Root ID: 66D53BE8A5056ECA!101
Remaining Free Space: 5368709070
Fetching details for OneDrive Root
OneDrive Root exists in the database
Initializing the Synchronization Engine ...
Syncing changes from OneDrive ...
Applying changes of Path ID: 66D53BE8A5056ECA!101
Uploading differences of ~/OneDrivePersonal
Processing root
The directory has not changed
Processing random_files
The directory has not changed
Processing random_images
The directory has not changed
Processing random_videos
The directory has not changed
Processing dummy_file.txt
The file has not changed
Processing qwer.txt
The file has not changed
Uploading new items of ~/OneDrivePersonal
Skipping item './my_pipe' as it is not a valid file
Applying changes of Path ID: 66D53BE8A5056ECA!101

@abraunegg abraunegg linked an issue Jul 6, 2020 that may be closed by this pull request
@abraunegg abraunegg added this to the v2.4.4 milestone Jul 6, 2020
@abraunegg abraunegg requested a review from norbusan July 6, 2020 20:25
docsmooth
docsmooth previously approved these changes Jul 6, 2020
Copy link

@docsmooth docsmooth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clear fix, tested as skipping pipes with the proper error message. I'm also noticing that the version with this fix causes cleaner messaging around bad symlinks, but i don't know if it's this fix.

@abraunegg
Copy link
Owner Author

@docsmooth

clear fix, tested as skipping pipes with the proper error message. I'm also noticing that the version with this fix causes cleaner messaging around bad symlinks, but i don't know if it's this fix.

Unsure what version you were previously running as your issue post did not include the client version. #942 fixed issues with symbolic links when they were relative links

* Align output messaging to be consistent with other 'skipping item' message output
@abraunegg
Copy link
Owner Author

Updated application output to be consistent:

Uploading new items of ~/OneDrivePersonal
Skipping item - excluded by skip_dir config: ./Dropbox/.dropbox.cache
Skipping item - excluded by skip_dir config: ./Images
Skipping item - excluded by skip_dir config: ./Music
Skipping item - excluded by skip_file config: ./file1.tmp
Skipping item - item is not a valid file: ./my_pipe
Skipping item - invalid symbolic link: ./wertsdeee.txt
Applying changes of Path ID: 66D53BE8A5056ECA!101
Syncing this OneDrive Personal Shared Folder: shared
Applying changes of Path ID: BC7D88EC1F539DCF!1288

@abraunegg abraunegg merged commit 8c0d431 into master Jul 7, 2020
@abraunegg abraunegg deleted the fix-issue-984 branch July 7, 2020 08:06
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators May 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OneDrive client does not skip (dead?) Unix pipes by default
2 participants