On 13.09.2019 11:04, Julian Foad wrote: > Branko Čibej wrote: >> On 11.09.2019 18:22, Julian Foad wrote: >>> $ svn ls https://archive.apache.org/dist >>> Redirecting to URL 'https://archive.apache.org/dist': >>> Redirecting to URL 'https://archive.apache.org/dist': >>> svn: E195019: Redirect cycle detected for URL >>> 'https://archive.apache.org/dist' >>> [...] >> >>> Does that make sense? >> >> It does not. >> >> [...] No DAV headers there, so it's not a Subversion repository and >> not our >> problem. Compare to: >> >> $ curl --request OPTIONS --include https://svn.apache.org/repos/asf > > Gah... I was comparing https://archive.apache.org/dist/ with > https://dist.apache.org/repos/dist/ and somehow it did not occur to me > that the former might not be a repository. > > Thanks for setting me straight! > > It is a confusing error message, though. The redirect "cycle" is only > Subversion that made it a cycle, and is not the real problem. We could > do better.
We could. Not canonicalizing the value of the Location header is an option; if it's not a valid URL, that's the server admin's problem, not ours (and as noted a number of times in other conversations, Subversion is not a browser so we don't have to worry about producing user-friendly URLs). Note that JavaHL has its own redirect-cycle detection (this is client-side stuff, not library-level stuff). -- Brane
