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.

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

Hotfix 2.0.x SUP-16159 Improve WebSocket error handling #1586

Closed
wants to merge 2 commits into from

Conversation

yrucrem
Copy link
Contributor

@yrucrem yrucrem commented Feb 7, 2024

Abstract

Errors from Vert.x when creating an eventbus WebSocket where not handled which caused a NullPointerException during event creation. Typical errors like invalid event addresses are now handled, and multiple registrations of the same address no longer lead to errors.

Checklist

General

  • Added abstract that describes the change
  • Added changelog entry to /CHANGELOG.adoc
  • Ensured that the change is covered by tests
  • Ensured that the change is documented in the docs

On API Changes

  • Checked if the changes are breaking or not
  • Added GraphQL API if applicable
  • Added Elasticsearch mapping if applicable

@yrucrem
Copy link
Contributor Author

yrucrem commented Feb 7, 2024

Will be upmerged from hotfix-1.10.x.

@yrucrem yrucrem closed this Feb 7, 2024
@yrucrem yrucrem deleted the hotfix-2.0.x-sup-16159 branch February 7, 2024 11:43
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.

1 participant