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

Support WebSocket connections #415

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bosunski
Copy link

No description provided.

Copy link

what-the-diff bot commented Jul 21, 2024

PR Summary

  • Improvements on HttpClient
    In the HttpClient file, changes allow the sendRequest application to now return the response object. In addition, the getBody() call is now type-hinted as DuplexStreamInterface, which provides a way to check if data is writable before establishing a connection.

  • Refinements in ProxyManager
    The ProxyManager file now incorporates the ResponseInterface from Psr\Http\Message. A new localRequestConnection variable is introduced and the required logic to handle it. There's also an enhanced mechanism for sending data through the proxy connection.

  • New Method in HttpConnection
    A new method, getConnection(), was added in HttpConnection file. This method centres around returning connection socket, thereby improving the efficiency of the server communication.

  • Data Sending functionality in TunnelMessageController
    In the TunnelMessageController file, an addition of a data event listener has been made to the httpConnection’s connection, paving the way for easier data transmission to the proxy.

  • Dependencies Update
    Finally, in the composer.json file, the version of the react/http package has been updated. A new dependency repository, https://github.com/phpsandbox/http, has also been added. This tune-up in dependencies helps to ensure that the application remains stable and secure with the latest libraries.

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