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

right way handle error for sendUTF method #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anchmelev
Copy link

Title: Improved Message Sending Handling in WebSocketTransport

Description:
The purpose of this pull request is to enhance the reliability and error handling of the send method in the WebSocketTransport class. The changes include making the send method return a promise, allowing the calling code more flexibility in managing the sending process and appropriately responding to different outcomes of the operation.

Key Changes:

  1. The send method now returns a promise, enabling the handling of success or error events of the sending process.
  2. Additional error handling and logging logic has been introduced, increasing the ease of debugging and monitoring.

These changes make the send method more robust and predictable in use, especially in scenarios that require confirmation of message delivery.

I request the review and approval of these changes as they improve the functionality and reliability of the WebSocketTransport.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant