fix(websocket): add error handling

[harshit2283]

SUMMARY

Adding Error handling in websocket to avoid crashing whole service due to a faulty socket etc

Certain Browsers (i.e Safari) don't play nicely with WS Protocol, they can end up crashing the whole service. ref

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[github-actions]

Congrats on making your first PR and thank you for contributing to Superset! 🎉 ❤️

We hope to see you in our Slack community too! Not signed up? Use our Slack App to self-register.

[villebro]

LGTM, thanks for the fix!

[villebro]

@harshit2283 it seems there's a linting error. Can you fix it so we can get this merged?

[harshit2283]

@harshit2283 it seems there's a linting error. Can you fix it so we can get this merged?

Have fixed it.

[villebro]

btw @harshit2283 as you appear to be using GAQ with websockets, it would be valuable to get your feedback on a recent proposal to simplify the GAQ architecture: #29515

[harshit2283]

@villebro Sure, I will check it out. Thank you!