Skip to content

Errors in SSE events should be caught #1646

@JammingBen

Description

@JammingBen

Initial Checklist

  • I understand this is a bug report and questions should be posted in the Community Discussions
  • I searched issues and couldn’t find anything (or linked relevant results below)

Bug Description

Errors happening in SSE events should be caught. It's probably not critical to throw them, but definitely cleaner to catch them to avoid further failures down the line.

This became apparent in #1551 (comment).

Reproduction Steps

This is hard to reproduce manually. The easiest would be to invalidate a request in the onSSEFolderCreatedEvent, then open 2 tabs of OC side by side and create a new folder in one of them.

Expected Outcome

The error in the other tab should be caught and logged.

Actual Outcome

The error gets thrown.

Metadata

Metadata

Assignees

Type

No type

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions