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.

Sign up for GitHub

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

Jump to bottom

[stable21] explicitly close source stream on local / encryption storage #29134

Merged
merged 2 commits into from
Oct 11, 2021
Merged

[stable21] explicitly close source stream on local / encryption storage #29134

merged 2 commits into from
Oct 11, 2021

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #28907

@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 21.0.6 milestone Oct 8, 2021
@kesselb kesselb self-assigned this Oct 8, 2021
@kesselb
explicitly close source stream on local storage
98d3836 
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
explicitly close source stream on encryption storage
b77b1d6 
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
Copy link
Contributor

kesselb commented Oct 8, 2021

Rebased the branch to have a recent CI run because we updated icewind/streams library.

@kesselb
Copy link
Contributor

kesselb commented Oct 11, 2021 •
edited
Loading

Web:

  • Upload a file
  • Upload an empty file
  • Upload eicar test file

Nautilus:

  • Upload a file
  • Upload an empty file
  • Upload eicar test file

@kesselb kesselb added the 3. to review Waiting for reviews label Oct 11, 2021
@juliusknorr juliusknorr merged commit e59f90d into stable21 Oct 11, 2021
@juliusknorr juliusknorr deleted the backport/28907/stable21 branch October 11, 2021 20:23
@blizzz blizzz mentioned this pull request Nov 3, 2021
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juliusknorr juliusknorr juliusknorr approved these changes

@kesselb kesselb kesselb approved these changes

@solracsf solracsf Awaiting requested review from solracsf

@blizzz blizzz Awaiting requested review from blizzz

@icewind1991 icewind1991 Awaiting requested review from icewind1991

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

@PVince81 PVince81 Awaiting requested review from PVince81

Assignees

@kesselb kesselb

Labels
3. to review Waiting for reviews bug
Projects
None yet
Milestone
Nextcloud 21.0.6
Development

Successfully merging this pull request may close these issues.
2 participants
@kesselb @juliusknorr