what's with all the redirects

[ctengel]

INFO: - "HEAD /bucket HTTP/1.1" 307 Temporary Redirect

[ctengel]

Ok so this is all around the locator checking to see if a bucket exists on a given server before redirecting a PUT there. If this check is not done it may result in a misdirected PUT and a 404.

The locator previously looked for the bucket without a trialing slash with a HEAD request. It would simply look for any non error as success. (2xx and 3xx)

The old system, inheriting from the Python simple server would take it either with or without the trailing slash and return 200 or 404 appropriately.

FastAPI brought in two differences. One is that ANY (existing or not) request to a bucket without a trailing slash gets a 307... Success! The other is that HEAD doesn't always work..405 - Fail!

Until recent commit the head would always return 3xx and apparently requests not bother doing a further head upon redirect (otherwise would have hit 4xx). This means that it return a redirect to any bucket even if it doesn't exist.

So we need to fix both of these and ideally do an efficient HEAD.

[ctengel]

interim untested fix is to HEAD the bucket with trailing slash, and for that HEAD to be implemented as GET without the body - performance enhancement maybe to make it faster, no need to render body

see b443210 and fastapi/fastapi#1773