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

DoorAttendant should accept more than TcpStreams #43

Open
robertdfrench opened this issue Feb 20, 2023 · 0 comments
Open

DoorAttendant should accept more than TcpStreams #43

robertdfrench opened this issue Feb 20, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@robertdfrench
Copy link
Owner

Is your feature request related to a problem? Please describe.
The only thing that can be sent through an application door is a TcpStream.

Describe the solution you'd like
Would like to be able to send data instead, or possibly data and a TcpStream. For example, the parsed headers could be sent as data, and the remaining TcpStream could be sent as a file descriptor so that the application door server can read it as the body. This will allow portunus to route the request based on the headers, but without copying the entire request body into its own memory first.

Describe alternatives you've considered
Alternatives are what is currently implented -- passing only a TcpStream. That may still be fine for the case where we want an unprivileged process to be able to handle streams without being able to bind to a port (are there any such cases?)

Additional context
Add any other context or screenshots about the feature request here.

@robertdfrench robertdfrench added the enhancement New feature or request label Feb 20, 2023
@robertdfrench robertdfrench self-assigned this Feb 20, 2023
@robertdfrench robertdfrench added this to the HTTP Routing milestone Feb 20, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant