HTTP store discards parameters in 301 redirects

[Dan Smith]

The HTTP store follows 301 redirects until it arrives at the actual image (or trips a check for max redirects). However, in each step of this loop, it copies only the path and discards the query portion. This can result in a failure to follow a redirect chain as expected.

In the cirros image we use in CI, the redirect chain ends up at a long github asset URI which contains tokens and other parameters, which must be provided or else a 400 is returned. The user also can't work around that by resolving the final URI and providing that, as the API has a 255 character limit for the store URI, and the one required by github exceeds that.

[Dan Smith (danms)]

Turns out this was just fixed after years, and is not yet released. Duplicates 1633860