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

What are we doing with IConnectionEndPointFeature #43786

Open
BrennanConroy opened this issue Sep 6, 2022 · 4 comments
Open

What are we doing with IConnectionEndPointFeature #43786

BrennanConroy opened this issue Sep 6, 2022 · 4 comments
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Milestone

Comments

@BrennanConroy
Copy link
Member

IConnectionEndPointFeature was added as part of bedrock in #10321.
But it is only implemented and exposed via DefaultConnectionContext which is only used by test projects in our repo.

We should figure out what to do with this feature.

@davidfowl
Copy link
Member

We should implement it

@wtgodbe wtgodbe added this to the Backlog milestone Sep 7, 2022
@ghost
Copy link

ghost commented Sep 7, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@davidfowl davidfowl modified the milestones: Backlog, .NET 8 Planning Sep 7, 2022
@davidfowl
Copy link
Member

I'll do it in .NET 8 planning 😄

@hexawyz
Copy link
Contributor

hexawyz commented Jul 24, 2023

I have a usecase where I would like to filter requests based on the endpoint used (using an IEndpointFilter), and it seems the IConnectionEndPointFeature would be a useful tool for this if it was available. (Testing on latest preview in the context of HTTP2/NamedPipe requests: it does not seem to be available, although the relevant information is present in internal classes)

Can we expect this for .NET 8.0, or is it too late?

@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 25, 2023
@amcasey amcasey modified the milestones: .NET 8 Planning, Backlog Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Projects
None yet
Development

No branches or pull requests

6 participants
@davidfowl @BrennanConroy @hexawyz @amcasey @wtgodbe and others