Skip to content

ConnectionAdapters have been removed from Kestrel #14905

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

Closed
Tratcher opened this issue Oct 2, 2019 · 0 comments · Fixed by #15044
Closed

ConnectionAdapters have been removed from Kestrel #14905

Tratcher opened this issue Oct 2, 2019 · 0 comments · Fixed by #15044
Assignees
Labels
breaking-change Indicates a .NET Core breaking change

Comments

@Tratcher
Copy link
Member

Tratcher commented Oct 2, 2019

ConnectionAdapters have been removed from Kestrel

As part of the move to move pubternal APIs to public, we've removed the concept of an IConnectionAdapter from Kestrel. Connection adapters are being replaced with connection middleware (very similar to http middleware in the ASP.NET Core pipeline but for lower level connections). HTTPS and Connection logging have been moved from connection adapters to connection middleware so those extension methods should continue to work seamlessly (but the guts have been re-implemented). See dotnet/aspnetcore#11412 for more details.

Discuss this issue on dotnet/aspnetcore#11475.

Version introduced

3.0

Old behavior

Kestrel extensibility components were created using IConnectionAdapter.

New behavior

Kestrel extensibility components are now created as middleware.

Reason for change

This is intended to provide a more flexible extensibility architecture.

Recommended action

Convert any implementations of IConnectionAdapter to use the new middleware pattern as shown here.

Category

ASP.NET Core

Affected APIs

IConnectionAdapter


Issue metadata

  • Issue type: breaking-change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change Indicates a .NET Core breaking change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants