Skip to content

skip e2e encrypted files with empty filename in metadata#5448

Merged
mgallien merged 1 commit intomasterfrom
bugfix/ignoreInvalidE2EMetadata
Feb 20, 2023
Merged

skip e2e encrypted files with empty filename in metadata#5448
mgallien merged 1 commit intomasterfrom
bugfix/ignoreInvalidE2EMetadata

Conversation

@mgallien
Copy link
Collaborator

we cannot decrypt a file without a name

for now we will ignore them

we should probably do much more for the user but I fail to see how to generate errors from e2ee module

@codecov
Copy link

codecov bot commented Feb 16, 2023

Codecov Report

Merging #5448 (a8a5dfe) into master (f6901fa) will decrease coverage by 0.15%.
The diff coverage is 0.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5448      +/-   ##
==========================================
- Coverage   57.97%   57.83%   -0.15%     
==========================================
  Files         141      141              
  Lines       18143    18146       +3     
==========================================
- Hits        10518    10494      -24     
- Misses       7625     7652      +27     
Impacted Files Coverage Δ
src/libsync/clientsideencryption.cpp 26.00% <0.00%> (-0.08%) ⬇️
src/libsync/vfs/cfapi/hydrationjob.cpp 51.56% <0.00%> (-3.65%) ⬇️
src/libsync/vfs/cfapi/vfs_cfapi.cpp 83.07% <0.00%> (-2.37%) ⬇️
src/libsync/vfs/cfapi/cfapiwrapper.cpp 73.18% <0.00%> (-1.82%) ⬇️
src/libsync/syncengine.cpp 80.90% <0.00%> (-0.43%) ⬇️

@mgallien mgallien force-pushed the bugfix/ignoreInvalidE2EMetadata branch from 0afc593 to 2e9952e Compare February 17, 2023 10:22
we cannot decrypt a file without a name

for now we will ignore them

we should probably do much more for the user but I fail to see how to
generate errors from e2ee module

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien force-pushed the bugfix/ignoreInvalidE2EMetadata branch from 2e9952e to a8a5dfe Compare February 20, 2023 08:50
@nextcloud-desktop-bot
Copy link

AppImage file: nextcloud-PR-5448-a8a5dfe6c607b9a7eecc5802d8841bdf6f1d9d27-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.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@mgallien mgallien merged commit 96dfa7a into master Feb 20, 2023
@mgallien mgallien deleted the bugfix/ignoreInvalidE2EMetadata branch February 20, 2023 17:49
@mgallien
Copy link
Collaborator Author

/backport to stable-3.7

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.

4 participants