Skip to content
This repository was archived by the owner on Dec 18, 2018. It is now read-only.

Move Int64 parsing logic to HttpAbstractions #1227

Closed
JunTaoLuo opened this issue Nov 22, 2016 · 2 comments
Closed

Move Int64 parsing logic to HttpAbstractions #1227

JunTaoLuo opened this issue Nov 22, 2016 · 2 comments
Assignees
@JunTaoLuo
Labels
3 - Done
Milestone
2.0.0-preview1

Comments

@JunTaoLuo
Copy link
Contributor

The logic in

KestrelHttpServer/src/Microsoft.AspNetCore.Server.Kestrel/Internal/Http/FrameHeaders.cs

Line 235 in 2940895

public static unsafe long ParseContentLength(StringValues value)
should be moved to Abstractions to allow other repos to take advantage of the performance boost.
@JunTaoLuo JunTaoLuo mentioned this issue Nov 22, 2016
Closed
@Tratcher
Copy link
Member

Especially request and response content-length headers

@JunTaoLuo JunTaoLuo added this to the 1.2.0 milestone Nov 22, 2016
@JunTaoLuo JunTaoLuo self-assigned this Nov 22, 2016
@JunTaoLuo JunTaoLuo mentioned this issue Nov 22, 2016
Merged
@JunTaoLuo JunTaoLuo modified the milestones: 2.0.0, 1.2.0 Nov 22, 2016
@JunTaoLuo
Copy link
Contributor Author

Done in https://github.com/aspnet/KestrelHttpServer/pull/1228/files

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@JunTaoLuo JunTaoLuo

Labels
3 - Done
Projects
None yet
Milestone
2.0.0-preview1
Development

No branches or pull requests
2 participants
@Tratcher @JunTaoLuo