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

Поиск
Список
Период
Сортировка
От Laurenz Albe
Тема Re: [PATCH] fix ICU explorer link in locale documentation
Дата
Msg-id 279934e1ab80d511d2739f840db1685d756ff99c.camel@cybertec.at
обсуждение исходный текст
Ответ на Re: [PATCH] fix ICU explorer link in locale documentation  (Anton Voloshin <a.voloshin@postgrespro.ru>)
Ответы Re: [PATCH] fix ICU explorer link in locale documentation  (Laurenz Albe <laurenz.albe@cybertec.at>)
Список 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




В списке pgsql-docs по дате отправления:

Предыдущее
От: "tanghy.fnst@fujitsu.com"
Дата:
Сообщение: RE: pg_type_d.h location incorrect
Следующее
От: 近藤雄太
Дата:
Сообщение: Re: Error building for 64-bit Windows (10)