Skip to content

[stable13] reset encryptionVersion to '1' if a file was stream copied - #9064

Merged
rullzer merged 1 commit into
stable13from
fix-copy-encrypted-files-stable13
Apr 4, 2018
Merged

[stable13] reset encryptionVersion to '1' if a file was stream copied #9064
rullzer merged 1 commit into
stable13from
fix-copy-encrypted-files-stable13

Conversation

@schiessle

Copy link
Copy Markdown
Member

backport of #9018

…his means that we basically write the file from scratch
Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
@schiessleschiessle added the 3. to review Waiting for reviews label Apr 3, 2018
@schiessleschiessle added this to the Nextcloud 13.0.2 milestone Apr 3, 2018
@schiessleschiessle changed the title reset encryptionVersion to '1' if a file was stream copied [stable13] reset encryptionVersion to '1' if a file was stream copied Apr 3, 2018
@codecov

codecovBot commented Apr 3, 2018

Copy link
Copy Markdown

Codecov Report

Merging #9064 into stable13 will increase coverage by <.01%.
The diff coverage is 71.42%.

@@ Coverage Diff @@## stable13 #9064 +/- ##
==============================================
+ Coverage 51.33% 51.33% +<.01% 
Complexity 25029 25029 ==============================================
Files 1609 1609 Lines 95305 95305 Branches 1376 1376 ==============================================
+ Hits 48924 48929 +5 + Misses 46381 46376 -5
Impacted FilesCoverage ΔComplexity Δ
lib/private/Files/Storage/Wrapper/Encryption.php68.24% <71.42%> (ø)145 <6> (ø)⬇️
apps/files_trashbin/lib/Expiration.php90.32% <0%> (-1.62%)29% <0%> (ø)
apps/files_trashbin/lib/Trashbin.php72.53% <0%> (+0.24%)136% <0%> (ø)⬇️
core/js/js.js66.29% <0%> (+0.55%)0% <0%> (ø)⬇️

@rullzer
rullzer merged commit e95184e into stable13Apr 4, 2018
@rullzer
rullzer deleted the fix-copy-encrypted-files-stable13 branch April 4, 2018 09:54
@MorrisJobkeMorrisJobke mentioned this pull request Apr 18, 2018
11 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@schiessle@rullzer@MorrisJobke