Add failing test to show missing headers in HEAD requests

RangerMauve / hypercore-fetch

Implementation of Fetch that uses the Hyper SDK for loading p2p content

MIT License

37 stars 13 forks source link

Closed josephmturner closed 1 year ago

RangerMauve commented 1 year ago

Some notes:

Content-Range should only be set if you have a Range header in the request, so by default it won't be in the response of a HEAD.

I'll add a test for Range requests to verify this.

Last-Modified should be getting set since the metadata for the file should be getting set, I'll check that too.

Content-Length not being set is defs a bug too.

I'll comment here once I have fixes

RangerMauve commented 1 year ago

Fixed in 9.0.7

josephmturner commented 1 year ago

It's working!! Thanks!!