On 23 March 2013 00:25, sebb <seb...@gmail.com> wrote: > On 23 March 2013 00:19, Jörg Schaible <joerg.schai...@gmx.de> wrote: >> sebb wrote: >> >>> On 22 March 2013 19:58, Gary Gregory <garydgreg...@gmail.com> wrote: >>>> I published the CSV site after your commit, but some links are still >>>> incorrect, for example: >>>> https://commons.apache.org/proper/releases/index.html instead of >>>> https://commons.apache.org/releases/index.html >>> >>> The problem is the redirect: >>> >>> https://commons.apache.org/csv/ >>> >>> redirects to >>> >>> https://commons.apache.org/proper/csv// >>> >>> The // at the end uses up one of the .. segments in the path. >>> >>> Try starting with >>> >>> https://commons.apache.org/csv >>> >>> or (what it redirects to) >>> >>> https://commons.apache.org/proper/csv/ >>> >>> Not sure yet how to fix the redirects. >> >> Fix the site. The URL in the BCEL POM should point to the new location. > > It does; the problem is in the redirect; working on it now.
I think csv is OK now. Please try it before we roll out the changes to the rest of the redirects. The original redirect looked like this: RedirectMatch ^/csv(.*) /proper/commons-csv/$1 So /csv => /proper/commons-csv/ and /csv/ => /proper/commons-csv// The latter is wrong. Thought about using: RedirectMatch ^/csv(.*) /proper/commons-csv/$1 but that would mean /csvx => /proper/commons-csvx which is wrong. Though of course it works for valid csv paths such as /csv/x => /proper/commons-csv/x So I ended up with: RedirectMatch ^/csv(/.*)? /proper/commons-csv$1 which is intended to ensure any path suffix after /csv starts with / Seems to work for me; please try and break it! --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
