Re: [PATCH 2/4] doc: update links to current pages
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Sep 22, 2023, 21:34 UTC
- Message-ID
- <CAPig+cT7G2G-PQ05cSAxLLoZPbz4eyHjPxUjMWhDB5ondyD0sA@mail.gmail.com>
- In-Reply-To
- <CACZqfqC_UwSPfn0f9L4TfkYDGU64t_en6=7wYMsnxA9fB2rSjw@mail.gmail.com>
On Fri, Sep 22, 2023 at 5:31 PM Josh Soref <jsoref@gmail.com> wrote:
Show 16 quoted lines
> Eric Sunshine wrote: > > > - * [2] http://json.org/ > > > + * [2] https://www.json.org/ > > > > Not sure why this change is needed considering that the simpler > > s/http/https/ seems sufficient. > > This is because that's their preference: > > ```sh > % curl -s http://json.org/|grep REFRESH > <meta HTTP-EQUIV="REFRESH" content="0; > url=https://www.JSON.org/json-en.html"></head> > ``` > > It's somewhat traditional to respect sites' self-identification.
It might be worth calling that out in the commit message of both patches if you happen to reroll so other reviewers understand the motivation. (That said, on its own, it's probably not worth a reroll.)