dotnet / dotnet/aspnetcore

Kestrel low-level should be in terms of ReadOnlySpan not Span

Open
#8,979 0 comments 0 reactions 0 assignees View on GitHub
affected-very-few area-networking enhancement severity-nice-to-have
Dominant language
C#
Stars
38.4k
Forks
10.9k
Avg merge
2d 5h
Merged PRs (30d)
276

Description

e.g.
```csharp
public partial interface IHttpHeadersHandler
{
void OnHeader(Span name, Span value);
void OnHeadersComplete();
}
```
should be
```csharp
public partial interface IHttpHeadersHandler
{
void OnHeader(ReadOnlySpan name, ReadOnlySpan value);
void OnHeadersComplete();
}
```
However, attention needs to be given to avoid any performance regressions.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.