Skip to content

Conversation

@backportbot-nextcloud
Copy link

Backport of #4968

@cypress
Copy link

cypress bot commented Nov 8, 2023

1 failed test on run #12340 ↗︎

1 149 2 0 Flakiness 0

Details:

[stable26] Fix 403 on close
Project: Text Commit: ee1c0cad0a
Status: Failed Duration: 03:19 💡
Started: Nov 9, 2023 9:12 AM Ended: Nov 9, 2023 9:16 AM
Failed  cypress/e2e/share.spec.js • 1 failed test

View
Output

Test Artifacts
Open test.md in viewer > Share a file with download disabled shows an error Screenshots

Review all test suite changes for PR #4969 ↗︎

@mejo- mejo- changed the title [stable26] [stable27] Fix 403 on close [stable26] Fix 403 on close Nov 8, 2023
@mejo-
Copy link
Member

mejo- commented Nov 8, 2023

/compile

mejo-
mejo- previously approved these changes Nov 8, 2023
@mejo- mejo- dismissed their stale review November 8, 2023 15:28

Failures seem related

@juliusknorr
Copy link
Member

So large failures seem unexpected, restarted

@juliusknorr
Copy link
Member

Yep other stable26 pr seems to proof differently #4972 (comment)

Somethings odd here

@juliusknorr
Copy link
Member

Base branch fix in #4973

y.js will send a last awareness update when closing the websocket.
Try to get this out before closing the sync service connection.

Signed-off-by: Max <[email protected]>
Signed-off-by: Jonas <[email protected]>
@mejo- mejo- force-pushed the backport/4968/stable26 branch from 39f8f7b to 7085dc6 Compare November 9, 2023 08:59
@mejo-
Copy link
Member

mejo- commented Nov 9, 2023

/compile

Signed-off-by: nextcloud-command <[email protected]>
@mejo- mejo- merged commit 1766df5 into stable26 Nov 9, 2023
@mejo- mejo- deleted the backport/4968/stable26 branch November 9, 2023 09:18
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.

5 participants