cloudflare / quiche

🥧 Savoury implementation of the QUIC transport protocol and HTTP/3
https://docs.quic.tech/quiche/
BSD 2-Clause "Simplified" License
9.4k stars 709 forks source link

quiche-server ignores POST request payloads #1401

Open ygina opened 1 year ago

ygina commented 1 year ago

The quiche-server currently ignores POST request payloads and decides the response based on headers alone (https://github.com/cloudflare/quiche/blob/master/apps/src/common.rs#L1555). It would be helpful for the server to properly consume and drain the stream for tests involving POST requests (#1400).

I started to implement this a while ago so feel free to reference but I don't think I can carry this PR through right now: https://github.com/cloudflare/quiche/compare/master...ygina:quiche:http-post?expand=1

LPardue commented 1 year ago

yeh we should modify the behaviour to make it easier to assess upload performance.