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

Add missing error handling for downloads of client side encryption #2967

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

FlexW
Copy link

@FlexW FlexW commented Mar 4, 2021

The missing error handlers can cause the sync to hang forever

Signed-off-by: Felix Weilbach [email protected]

Copy link
Member

@er-vin er-vin left a comment

Choose a reason for hiding this comment

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

Good catch! Totally overlooked that one last time I was looking at this. A little thing needs to be adjusted IMHO.

src/libsync/propagatedownloadencrypted.cpp Outdated Show resolved Hide resolved
src/libsync/propagatedownloadencrypted.cpp Outdated Show resolved Hide resolved
src/libsync/propagatedownloadencrypted.cpp Show resolved Hide resolved
src/libsync/propagatedownloadencrypted.h Outdated Show resolved Hide resolved
@FlexW FlexW force-pushed the bugfix/fix-missing-error-handling-download-encrypted branch from e89886e to a67c73d Compare March 10, 2021 09:29
@FlexW
Copy link
Author

FlexW commented Mar 11, 2021

/rebase

@github-actions github-actions bot force-pushed the bugfix/fix-missing-error-handling-download-encrypted branch from a67c73d to 241ff1e Compare March 11, 2021 09:43
@FlexW
Copy link
Author

FlexW commented Mar 11, 2021

/rebase

The missing error handlers can cause the sync to hang forever

Signed-off-by: Felix Weilbach <[email protected]>
@github-actions github-actions bot force-pushed the bugfix/fix-missing-error-handling-download-encrypted branch from 241ff1e to 88426de Compare March 11, 2021 10:59
@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-2967-88426de4ddf18d9a6b78200bf21885df68214c5e-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@FlexW FlexW merged commit 72858da into master Mar 11, 2021
@FlexW FlexW deleted the bugfix/fix-missing-error-handling-download-encrypted branch March 11, 2021 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants