Re: [PATCH 1/1] snapshot::web Fix internal redirects to farm

[Philipp Kern <pkern@debian.org>]

Hi,

On 11/15/24 2:27 PM, Felix Moessbauer wrote:
> We already have a rule to let Varnish follow the 302 redirects from the
> /archive endpoint to the farm (/file) endpoint. This rule was not
> effective, though, as it assumed that the 302 redirects are absolute
> URIs (including the protocol and host part). We now change this to
> follow the realtive (internal) redirects, restoring this behavior.
> 
> A nice side-effect is that the artifacts are now delivered with the
> correct file-name (instead of the farm hash). Further it effectively
> cuts the number of http requests to s.d.o in half.
I tried, but this caused Bad Requests to be emitted by Apache.

Kind regards
Philipp Kern