Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adding websocket upgrade support #795

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

Conversation

taiidani
Copy link

Taking inspiration from #637 and attempting to address the changes requested from the PR review

Adding a handler for an "upgrade" request so that the proxy properly manages the connection to the downstream.

Relevant issues

Fixes #142.

Contributor checklist
  • Provide tests for the changes (unless documentation-only)
  • Documented any new features, CLI switches, etc. (if applicable)
    • Server --help output
    • README.md
    • doc/http-server.1 (use the same format as other entries)
  • The pull request is being made against the master branch
Maintainer checklist
  • Assign a version triage tag
  • Approve tests if applicable

@github-actions
Copy link

github-actions bot commented Feb 8, 2023

This pull request has been inactive for 360 days

@github-actions github-actions bot added the stale label Feb 8, 2023
@taiidani taiidani marked this pull request as draft May 14, 2024 16:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WebSockets not supported in proxy mode
1 participant