Skip to content

Conversation

@icewind1991
Copy link
Member

While we do try to close the stream before the end-of-request cleanup, in cases where this isn't done for any reason we shouldn't error

@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Aug 31, 2022
@icewind1991 icewind1991 added this to the Nextcloud 25 milestone Aug 31, 2022
@icewind1991 icewind1991 requested review from a team, CarlSchwan, PVince81 and juliusknorr and removed request for a team August 31, 2022 13:08
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍 so you found it 😄

@PVince81 PVince81 added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 31, 2022
@blizzz blizzz mentioned this pull request Sep 1, 2022
@PVince81 PVince81 merged commit 7ea8427 into master Sep 1, 2022
@PVince81 PVince81 deleted the hash-wrapper-closed-hash branch September 1, 2022 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants