Den tis 28 dec. 2021 kl 21:14 skrev <dsahlb...@apache.org>: > Author: dsahlberg > Date: Tue Dec 28 20:14:17 2021 > New Revision: 1896485 > > URL: http://svn.apache.org/viewvc?rev=1896485&view=rev > Log: > In site/staging: The ASF download script currently return the same site > for both "http" and "backup" (after the migration to dlcdn.a.o). > > * download.html: Avoid displaying the same URL twice in the <select> > > Modified: > subversion/site/staging/download.html > > Modified: subversion/site/staging/download.html > URL: > http://svn.apache.org/viewvc/subversion/site/staging/download.html?rev=1896485&r1=1896484&r2=1896485&view=diff > > ============================================================================== > --- subversion/site/staging/download.html (original) > +++ subversion/site/staging/download.html Tue Dec 28 20:14:17 2021 > @@ -73,9 +73,9 @@ Other mirrors: > [if-any ftp] > [for ftp]<option value="[ftp]"[is ftp preferred] > selected="selected"[end]>[ftp]</option>[end] > [end] > - [if-any backup] > + [if-any backup][is backup http][else][# Only show backup if different > from http ] > [for backup]<option value="[backup]"[is backup preferred] > selected="selected"[end]>[backup] (backup)</option>[end] > - [end] > + [end][end] > </select> > <input type="submit" value="Change" /> > </form> >
The reasoning behind this change is described in the log message but I'm not happy about it for a few reasons: * [http] and [backup] are arrays, so comparing these are probably not a good idea. I can't come up with anything reasonably simple though. * The lua version of EZT doesn't seem to support [else] at all. The fact that it works for me is probably purely accidental. I have a strong feeling that the download page will not work in case different URLs are returned from the "closer.lua" script. Please check if you are getting different lists in [http] and [backup] from https://subversion-staging.apache.org/download.cgi?as_json=1 (I'm getting https://dlcdn.apache.org/ on both of these). In that case check if the <select> looks reasonable at https://subversion-staging.apache.org/download.cgi. Also compare with our published download page at https://subversion.apache.org/download.cgi. I'm inclined to revert this change and live with the fact that the download page can look slightly silly (displaying dlcdn.a.o as both the main and the backup mirror) until ASF Infra check on closer.lua (or someone else find a more clever way of writing the EZT comparison code). Kind regards, Daniel
