Re: [PATCH] fix ICU explorer link in locale documentation

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>
Cc: Anton Voloshin <a(dot)voloshin(at)postgrespro(dot)ru>, Magnus Hagander <magnus(at)hagander(dot)net>, Pg Docs <pgsql-docs(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] fix ICU explorer link in locale documentation
Date: 2021-09-27 18:31:03
Message-ID: 20210927183103.GA12052@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs

On Fri, Sep 3, 2021 at 09:39:45PM +0200, Laurenz Albe wrote:
> On Thu, 2021-05-27 at 08:59 +0200, Laurenz Albe wrote:
> > On Thu, 2021-05-27 at 11:10 +0700, Anton Voloshin wrote:
> > > So I guess the https://icu4c-demos.unicode.org/icu-bin/locexp link is
> > > still the proper one, and eventually it will be fixed. Anyway, the old
> > > link, https://ssl.icu-project.org/icu-bin/locexp, if one goes through
> > > "wrong domain in certificate" errors, still redirects to the same
> > > https://icu4c-demos.unicode.org/icu-bin/locexp
> > >
> > > I believe changing link in Postgres' docs is better than keeping the old
> > > one, even though they are currently both 404 one way or another. At
> > > least it would save people following that link from scary "Warning:
> > > Potential Security Risk Ahead" messages before properly 404-ing.
> > >
> > > [1]
> > > https://sourceforge.net/p/icu/mailman/icu-support/thread/CAD%2BuSKi%3DQeLWiivJmaFBHPg-tH9PjRriTOXxi9t2NQGr3VONww%40mail.gmail.com/#msg37261067
> >
> > I agree that the documentation link should be changed.
> >
> > There are two options:
> >
> > 1. Trust that they will fix it soon and change the link to
> >    https://icu4c-demos.unicode.org/icu-bin/locexp
> >
> > 2. Despair and remove the link from the documentation.
> >
> > I suggest that we wait until the end of the June 2021 commitfest and decide then.
> > If it is still not fixed then, the second option is better - or we move it to
> > the next commitfest.
>
> It is way past that time, and the link is still broken.
>
> So I guess it would be the best to remove the link from the documentation.

Should we change it to a link to the locexp source?

https://github.com/unicode-org/icu-demos/tree/main/locexp

This is the source for the locale explorer CGI which is on IBM's
external website at:

http://demo.icu-project.org/icu-bin/locexp

--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EDB https://enterprisedb.com

If only the physical world exists, free will is an illusion.

In response to

Responses

Browse pgsql-docs by date

  From Date Subject
Next Message Michael Paquier 2021-09-29 02:57:58 Re: Minor doc fixes
Previous Message Laurenz Albe 2021-09-27 11:35:27 Re: Missing mention of autovacuum_work_mem