Re: [PATCH v2 4/4] doc: refer to internet archive
- From
Elijah Newren <newren@gmail.com>
- Date
- Nov 24, 2023, 15:32 UTC
- Message-ID
- <CABPp-BGVFjdRhtXtm0Wgygu3yrxzDfZG2vxyqHqiFzfQNnOxTQ@mail.gmail.com>
- In-Reply-To
- <CACZqfqCksg=uMh=URh-0KEoPA9cvjgDAKPhn+0WZti-y2A540Q@mail.gmail.com>
On Thu, Nov 23, 2023 at 7:24 PM Josh Soref <jsoref@gmail.com> wrote:
Show 20 quoted lines
> > Elijah Newren wrote: > > Thanks, these all look good, except on of the old links works for me. > > Maybe it was just down the day you checked? > > > More comments on that below... > > > > - # Atom: http://www.atomenabled.org/developers/syndication/ > > > - # RSS: http://www.notestips.com/80256B3A007F2692/1/NAMO5P9UPQ > > > + # Atom: https://web.archive.org/web/20230815171113/https://www.atomenabled.org/developers/syndication/ > > > + # RSS: https://web.archive.org/web/20030729001534/http://www.notestips.com/80256B3A007F2692/1/NAMO5P9UPQ > > > > The original www.atomenabled.org link works for me. > > That's odd. As you can see based on my archive.org link, at some point > they had `https` support (their certificate expired Feb 7, 2021 UTC). > > Personally, as a former web browser developer, I'd rather people rely > on archived authenticated content than content that could be rewritten > in transit.
I think that'd be perfectly reasonable, I was just thrown because the justification was that the links no longer worked. Mentioning why the changes are being made in the commit message, and in particular that there are two reasons for the changes, would have been sufficient.
> That said, I'm going to drop this change.
Your call.