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

From: Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>
To: Anton Voloshin <a(dot)voloshin(at)postgrespro(dot)ru>, Magnus Hagander <magnus(at)hagander(dot)net>
Cc: Pg Docs <pgsql-docs(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] fix ICU explorer link in locale documentation
Date: 2021-05-27 06:59:53
Message-ID: 279934e1ab80d511d2739f840db1685d756ff99c.camel@cybertec.at
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs

On Thu, 2021-05-27 at 11:10 +0700, Anton Voloshin wrote:
> On 18/05/2021 03:46, Magnus Hagander wrote:
> > https://icu4c-demos.unicode.org/icu-bin/locexp generates a 404 for me now.
> >
> > This might be something temporary though, because AFAICT it's where
> > you end up when you follow the links from the ICU git repo...
>
> Yes, same 404 for me. This error has been reported on icu-issues mailing
> list on April 13th [1] and the reply from Stephan Pakebusch was:
>
> > This demo has been out of date (by several releases) and is being updated for ICU 69.
> > In the meantime there was an issue with this particular demo.
> > We’re looking into the problem you noted.
> >
> > I’m not sure if there’s a tracking issue for this specific, but you can follow the development on https://github.com/unicode-org/icu-demos
>
> 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.

Yours,
Laurenz Albe

In response to

Responses

Browse pgsql-docs by date

  From Date Subject
Next Message Laurenz Albe 2021-05-27 07:45:27 Re: [PATCH] add link to domain data types section from locale documentation
Previous Message tanghy.fnst@fujitsu.com 2021-05-27 06:13:36 RE: pg_type_d.h location incorrect