Fix Content-Range response when body size equals expected range length

laminas / laminas-httphandlerrunner

Execute PSR-15 RequestHandlerInterface instances and emit responses they generate.

https://docs.laminas.dev/laminas-httphandlerrunner/

BSD 3-Clause "New" or "Revised" License

63 stars 20 forks source link

Fix Content-Range response when body size equals expected range length #28

Open ajgarlag opened 2 years ago

ajgarlag commented 2 years ago

Signed-off-by: Antonio J. García Lagar ajgarcia@c17.net

Q	A
Documentation	no
Bugfix	yes
BC Break	no
New Feature	no
RFC	no
QA	no

Description

When the response body has already been limited to the requested range, the emitter should ignore the Content-Range header and emit the full response body.

This is an alternative to #23 to fix #22

boesing commented 2 years ago

This looks pretty good. It does not modify existing behavior I guess (at least not a behavior which is covered by tests). I guess we can supersede #23 for this?