Date: Mon, 27 Sep 2021 14:31:03 -0400
From: Bruce Momjian <bruce@momjian.us>
To: Laurenz Albe <laurenz.albe@cybertec.at>
Cc: Anton Voloshin <a.voloshin@postgrespro.ru>,
	Magnus Hagander <magnus@hagander.net>,
	Pg Docs <pgsql-docs@lists.postgresql.org>
Subject: Re: [PATCH] fix ICU explorer link in locale documentation
Message-ID: <20210927183103.GA12052@momjian.us>
References: <68f32992-7ba7-f797-2bb3-8851bd70d95d@postgrespro.ru>
 <CABUevEzePbHCk75uzzpRwLbv0Evra1sFt3YAi=ahtmkADQ0tHw@mail.gmail.com>
 <ca09a8ad-5361-4e5e-4e13-4017fa593bb0@postgrespro.ru>
 <279934e1ab80d511d2739f840db1685d756ff99c.camel@cybertec.at>
 <865474b6ab538cdd9419672c4bf2aba47ee3e9f3.camel@cybertec.at>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <865474b6ab538cdd9419672c4bf2aba47ee3e9f3.camel@cybertec.at>
User-Agent: Mutt/1.10.1 (2018-07-13)
Archived-At: 
 <https://www.postgresql.org/message-id/20210927183103.GA12052%40momjian.us>
Precedence: bulk

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@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

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