Enable HEAD request instead of ony GET#269
Conversation
abkfenris
left a comment
There was a problem hiding this comment.
@wachsylon this looks good to me, let me know when you are happy with it and pre-commit is happy. Codecov seems to be a little nutty at the moment, so we can ignore it.
|
@wachsylon, can you give some examples of what zarr apps need to be making HEAD requests? I'm struggling to see how this aligns with the classic call patterns against HTTP Zarr stores. |
|
i was a bit unspecific: I think that this specific app first tries to find out whether there really is sth to GET before it actually gets it.
so rather "in addition" than "instead of GET".
I needed it for a zarr-js app.
I thought that we just do not want xpublish to return "method not allowed" for HEADs anyway, no matter what the standard actually is. |
Some ZARR-applications use HEAD requests instead of GET requests. Found out that FASTAPI returns "Method not allowed" for HEAD requests where only GET requests are defined. This would enable it by returning the same as the GETs.