-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support HEAD requests #170
Comments
What use-case is this going to serve for you? Since Frisbii takes a no-buffering approach I think the utility would be limited to finding out if the first block is available or not. Is that all you need from it? I know Boxo takes a different approach and does a bunch of up-front buffering before it starts serving so can deal with a few other classes of errors but I'm not very keen on that approach. |
We're currently using a deployed Frisbii instance as an SP we can rely on, in automated tests. Therefore for us it would be sufficient if the HEAD request just always got a 200 response, for example. It also doesn't need to have any headers. Since our use case is limited, I think we could also live off a fork (I have a very basic HEAD implementation ready locally). Wdyt? |
@rvagg doing "quick root block presence check" for HEAD is sensible. Boxo and Storacha already do that as optimization for $ curl "https://dag.w3s.link/ipfs/QmbWqxBEKC3P8tqsKc98xmWNzrzDtRLMiMPL8wBuTGsMnR?format=raw" -I
HTTP/2 200
$ curl "https://ipfs.io/ipfs/QmbWqxBEKC3P8tqsKc98xmWNzrzDtRLMiMPL8wBuTGsMnR?format=raw" -I
HTTP/2 200 Implementation wise, HEAD does not have to fetch entire payload nor return any extra headers GET does based on payload (RFC uses SHOULD, but not MUST). This enables us to return HTTP 200/404 fast. Our focus is on FYSA HEAD is basic presence check will improve performance of HTTP retrieval, we started work across IPFS ecosystem:
|
Thanks for all the details @lidel, this is helpful. I did see ipfs/specs#491 pass by but how much of a limbo is that in? Is it likely to land roughly as is or should I wait and watch progress on that? |
@rvagg it will land :) I hate to work on things that don't land. (and it will not change much). |
This issue where @hsanjuan requested Spark to check HEAD retrievals can offer some additional context: |
From https://specs.ipfs.tech/http-gateways/trustless-gateway/#head-ipfs-cid-path-params:
The text was updated successfully, but these errors were encountered: