Andrew Casey
Andrew Casey
> I don't think we need a switch. If this lands in a mid .NET 9 preview, then it should go through a lot of use and testing before GA....
@ladeak Did you receive the initial feedback you needed? Is this ready for a full review or are you still working on it. There's no rush - I just wondered...
> hardcode a "big enough" limit or to expose this on Kestrel options under the Http2 limits? what default value should this limit have? Since the spec doesn't give a...
> A question if it is public: should it apply to a single header or to the total headers. Consumers HttpClient and Edge had a total while curl per header...
Thanks for the prototype and the thoughtful write-up. > @amcasey , I added a commit that has a new limit on `KestrelServerLimits`, and this is also respected by `Http2FrameWriter`. I...
I was actually thinking of a single check for _each_ protocol, but I'm fine with merging the checks if that's an option.
> Each piece - the reason I keep coming back to this idea is because my understanding was that the problem is allocating a really large buffer, and this would...
Thanks! I agree that an additional iteration would be undesirable. I'm going to be out of town next week, but @mgravell should be around to answer your questions (except Monday,...
@ladeak I still feel like I'm missing something fundamental. It seems like the input to the writer is a collection of header names and values. Is that correct? Maybe it's...
While I was out, I had some chance to look at the HTTP/2 spec and noticed that [8.2.3](https://datatracker.ietf.org/doc/html/rfc9113#section-8.2.3) gives us another way to split up long headers. Do you happen...