No, it's correct.

Almost all Apache projects use dlcdn for the binary artifacts (I think
the page you are referencing is not up to date for artifacts). For
instance:

https://maven.apache.org/download.cgi
https://tomcat.apache.org/download-10.cgi
etc

However, checksum and signature should go via downloads.apache.org
(not dlcdn even if it works).

Instead of revert, I will simply fix signature and checksum download URLs.

If you prefer I revert, fair enough.

Regards
JB



On Tue, Mar 29, 2022 at 6:17 PM Robbie Gemmell <robbie.gemm...@gmail.com> wrote:
>
> This seems wrong, I think this should be reverted.
>
> We are still asked to use the redirect scripts so that infra can
> manage resources as needed, and are meant to link to
> downloads.apache.org for checksums as the page did.
> https://infra.apache.org/release-download-pages.html
>
> The old links also redirected to the archive whenever the release is
> removed from the CDN, whereas your replacement ones will 404.
> Meanwhile as you didnt update all the release template bits there are
> now 2 different styles in use across the site (or even in the same
> page in cases) as all the other generated bits are still as they were
> before.
>
> Robbie
>
> On Mon, 21 Mar 2022 at 13:49, <jbono...@apache.org> wrote:
> >
> > This is an automated email from the ASF dual-hosted git repository.
> >
> > jbonofre pushed a commit to branch main
> > in repository https://gitbox.apache.org/repos/asf/activemq-website.git
> >
> >
> > The following commit(s) were added to refs/heads/main by this push:
> >      new fc6da54  Use "new" dlcdn download URLs
> > fc6da54 is described below
> >
> > commit fc6da54d8ff0f5972364a249803aa521a5ff4105
> > Author: Jean-Baptiste Onofré <jbono...@apache.org>
> > AuthorDate: Mon Mar 21 14:49:28 2022 +0100
> >
> >     Use "new" dlcdn download URLs
> > ---
> >  src/components/artemis/download/index.md                | 2 +-
> >  src/components/classic/download/index.md                | 6 +++---
> >  src/components/cms/developers/creating-distributions.md | 8 +++-----
> >  3 files changed, 7 insertions(+), 9 deletions(-)
> >
> > diff --git a/src/components/artemis/download/index.md 
> > b/src/components/artemis/download/index.md
> > index 07fe8bd..dcdfad0 100644
> > --- a/src/components/artemis/download/index.md
> > +++ b/src/components/artemis/download/index.md
> > @@ -31,7 +31,7 @@ This is the current release of ActiveMQ Artemis Native, 
> > which is a sub component
> >
> >  [Release Notes](release-notes-native-1.0.2)
> >
> > -Source 
> > Distribution:|[apache-artemis-native-1.0.2-source-release.zip](https://www.apache.org/dyn/closer.cgi?filename=activemq/activemq-artemis-native/1.0.2/activemq-artemis-native-1.0.2-source-release.zip&action=download)|[SHA512](https://downloads.apache.org/activemq/activemq-artemis-native/1.0.2/activemq-artemis-native-1.0.2-source-release.zip.sha512)|[GPG
> >  
> > Signature](https://downloads.apache.org/activemq/activemq-artemis-native/1.0.2/activemq-artemis-native-1.0.2-source-release.zip.asc)|
> > +Source 
> > Distribution:|[apache-artemis-native-1.0.2-source-release.zip](https://dlcdn.apache.org/activemq/activemq-artemis-native/1.0.2/activemq-artemis-native-1.0.2-source-release.zip)|[SHA512](https://dlcdn.apache.org/activemq/activemq-artemis-native/1.0.2/activemq-artemis-native-1.0.2-source-release.zip.sha512)|[GPG
> >  
> > Signature](https://dlcdn.apache.org/activemq/activemq-artemis-native/1.0.2/activemq-artemis-native-1.0.2-source-release.zip.asc)|
> >
> >  This is the native layer used by ActiveMQ Artemis for storage. The broker 
> > binary archives above already include a pre-compiled version of this 
> > component.
> >  <br/>
> > diff --git a/src/components/classic/download/index.md 
> > b/src/components/classic/download/index.md
> > index 58321ca..78e7953 100644
> > --- a/src/components/classic/download/index.md
> > +++ b/src/components/classic/download/index.md
> > @@ -21,9 +21,9 @@ It is important to [verify the 
> > integrity](#verify-the-integrity-of-downloads) of
> >  **NOTE**: This is the last planned 5.15.x release. Users should upgrade to 
> > the current 5.16.x stream for ongoing releases.
> >  {% endif %}
> >
> > -Windows|[apache-activemq-{{release.version}}-bin.zip](https://www.apache.org/dyn/closer.cgi?filename=/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.zip&action=download)|[SHA512](https://downloads.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.zip.sha512)|[GPG
> >  
> > Signature](https://downloads.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.zip.asc)
> > -Unix/Linux/Cygwin|[apache-activemq-{{release.version}}-bin.tar.gz](https://www.apache.org/dyn/closer.cgi?filename=/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.tar.gz&action=download)|[SHA512](https://downloads.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.tar.gz.sha512)|[GPG
> >  
> > Signature](https://downloads.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.tar.gz.asc)
> > -Source Code 
> > Distribution:|[activemq-parent-{{release.version}}-source-release.zip](https://www.apache.org/dyn/closer.cgi?filename=/activemq/{{release.version}}/activemq-parent-{{release.version}}-source-release.zip&action=download)|[SHA512](https://downloads.apache.org/activemq/{{release.version}}/activemq-parent-{{release.version}}-source-release.zip.sha512)|[GPG
> >  
> > Signature](https://downloads.apache.org/activemq/{{release.version}}/activemq-parent-{{release.version}}-source-release.zip.asc)
> > +Windows|[apache-activemq-{{release.version}}-bin.zip](https://dlcdn.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.zip)|[SHA512](https://dlcdn.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.zip.sha512)|[GPG
> >  
> > Signature](https://dlcdn.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.zip.asc)
> > +Unix/Linux/Cygwin|[apache-activemq-{{release.version}}-bin.tar.gz](https://dlcdn.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.tar.gz)|[SHA512](https://dlcdn.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.tar.gz.sha512)|[GPG
> >  
> > Signature](https://dlcdn.apache.org/activemq/{{release.version}}/apache-activemq-{{release.version}}-bin.tar.gz.asc)
> > +Source Code 
> > Distribution:|[activemq-parent-{{release.version}}-source-release.zip](https://dlcdn.apache.org/activemq/{{release.version}}/activemq-parent-{{release.version}}-source-release.zip)|[SHA512](https://dlcdn.apache.org/activemq/{{release.version}}/activemq-parent-{{release.version}}-source-release.zip.sha512)|[GPG
> >  
> > Signature](https://dlcdn.apache.org/activemq/{{release.version}}/activemq-parent-{{release.version}}-source-release.zip.asc)
> >              {% break %}
> >          {% endif %}
> >      {% endfor %}
> > diff --git a/src/components/cms/developers/creating-distributions.md 
> > b/src/components/cms/developers/creating-distributions.md
> > index c50ba7e..d3ea662 100644
> > --- a/src/components/cms/developers/creating-distributions.md
> > +++ b/src/components/cms/developers/creating-distributions.md
> > @@ -18,9 +18,7 @@ Under the source dir, we just drop the archive files 
> > (*.zip, *.tar.gz, *.zip.asc
> >
> >  The links on our download pages reference an apache CGI script to handle 
> > the mirrors. This is a quick and dirty way of doing this, but it works 
> > until we come up with a better way. An example usage of the link is the url 
> > below:
> >
> > -[http://www.apache.org/dyn/closer.cgi/activemq/activemq-cpp/source/activemq-cpp-1.1-src.zip](http://www.apache.org/dyn/closer.cgi/activemq/activemq-cpp/source/activemq-cpp-1.1-src.zip)
> > -
> > -The cgi script "closer.cgi" takes a file resource relative to 
> > www/www.apache.org/dist and generates a download page for it with the list 
> > of mirrors.
> > +[https://dlcdn.apache.org/activemq/activemq-cpp/source/activemq-cpp-1.1-src.zip](https://dlcdn.apache.org/activemq/activemq-cpp/source/activemq-cpp-1.1-src.zip)
> >
> >  To see this in action, go to the amq-cpp 1.1 download page here: 
> > [activemq-cpp-11-release.md](../../download/11-release)
> >
> > @@ -62,9 +60,9 @@ The page [ActiveMQ-CPP product version 
> > number](Index/Overview/Download/ActiveMQ-
> >      ````
> >  *   Copy the distribution files to 
> > **/www/www.apache.org/dist/activemq/activemq-cpp/source** on minotaur.
> >  *   Wait 24 hours for the mirrors to be updated with the distribution.
> > -*   Update the links on the download page to reference your distribution 
> > through the CGI script. This simply means that you precede the file names 
> > with the path 
> > [http://www.apache.org/dyn/closer.cgi/activemq/activemq-cpp/source/](http://www.apache.org/dyn/closer.cgi/activemq/activemq-cpp/source/).
> > +*   Update the links on the download page to reference your distribution 
> > through the CGI script. This simply means that you precede the file names 
> > with the path 
> > [https://dlcdn.apache.org/activemq/activemq-cpp/source/](https://dlcdn.apache.org/activemq/activemq-cpp/source/).
> >      ```
> > -    i.e. 
> > http://www.apache.org/dyn/closer.cgi/activemq/activemq-cpp/source/activemq-cpp-1.1-src.zip
> > +    i.e. 
> > https://dlcdn.apache.org/activemq/activemq-cpp/source/activemq-cpp-1.1-src.zip
> >      ```
> >  *   Generate the Doxygen API and place it under 
> > /www/activemq.apache.org/cms/api_docs.
> >  *   Add the link to the release API to the 
> > [API](Index/Site/NavigationIndex/Site/Navigation/Index/Site/Navigation/api.md)
> >  page.

Reply via email to