Re: Redirect cycle detected for URL, if redirect adds a trailing slash

2019-09-13 Thread Branko Čibej
On 13.09.2019 14:25, Julian Foad wrote: > Branko Čibej wrote: >> It looks correct to me, and the result you report against >> archive.apache.org is correct, too. I'd say, go ahead and commit it, >> we'll soon see if something goes wrong because of that. > > Ack. http://svn.apache.org/r1866899 > >>

Re: Redirect cycle detected for URL, if redirect adds a trailing slash

2019-09-13 Thread Julian Foad
Branko Čibej wrote: It looks correct to me, and the result you report against archive.apache.org is correct, too. I'd say, go ahead and commit it, we'll soon see if something goes wrong because of that. Ack. http://svn.apache.org/r1866899 The only thing I can think of is that we *require* tha

Re: Redirect cycle detected for URL, if redirect adds a trailing slash

2019-09-13 Thread Branko Čibej
On 13.09.2019 13:36, Julian Foad wrote: > Branko Čibej wrote: >>> 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

Re: Redirect cycle detected for URL, if redirect adds a trailing slash

2019-09-13 Thread Julian Foad
Branko Čibej wrote: 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 admi

Re: Redirect cycle detected for URL, if redirect adds a trailing slash

2019-09-13 Thread Branko Čibej
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

Re: Redirect cycle detected for URL, if redirect adds a trailing slash

2019-09-13 Thread Julian Foad
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' [...] >

Re: Redirect cycle detected for URL, if redirect adds a trailing slash

2019-09-11 Thread Branko Čibej
On 11.09.2019 18:22, Julian Foad wrote: > [[[ > $ curl --head https://archive.apache.org/dist > HTTP/1.1 301 Moved Permanently > Location: https://archive.apache.org/dist/ > ... > > $ svn ls https://archive.apache.org/dist > Redirecting to URL 'https://archive.apache.org/dist': > Redirecting to URL

Redirect cycle detected for URL, if redirect adds a trailing slash

2019-09-11 Thread Julian Foad
[[[ $ curl --head https://archive.apache.org/dist HTTP/1.1 301 Moved Permanently Location: https://archive.apache.org/dist/ ... $ 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: Redire