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
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 |