public inbox for [email protected]  

help / color / mirror / Atom feed

From: Laurenz Albe <[email protected]>
To: Anton Voloshin <[email protected]>
To: Magnus Hagander <[email protected]>
Cc: Pg Docs <[email protected]>
Subject: Re: [PATCH] fix ICU explorer link in locale documentation
Date: Fri, 03 Sep 2021 21:39:45 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<CABUevEzePbHCk75uzzpRwLbv0Evra1sFt3YAi=ahtmkADQ0tHw@mail.gmail.com>
	<[email protected]>
	<[email protected]>

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

Yours,
Laurenz Albe

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: [PATCH] fix ICU explorer link in locale documentation
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox