Обсуждение: Avoiding redirects on internal links

Поиск
Список
Период
Сортировка

Avoiding redirects on internal links

От
Daniel Gustafsson
Дата:
href=“/foo/bar” will 301 redirect to /foo/bar/, and we have a few of those
links left.  The attached patch updates all the links that I could find.
Unless there are objections I intend to commit that, but I wouldn’t mind an
extra set of eyes on it.  I did skip the presskits as they are unlikely to see
much traffic once 11 ships, so it didn’t seem worth the effort.

The patch also changes all /community/lists/ links to /list/ to avoid yet
another 301 redirect on the old URL, and removes an anchor on a list URL which
doesn’t exist.

cheers ./daniel


Вложения

Re: Avoiding redirects on internal links

От
"Jonathan S. Katz"
Дата:
Hi Daniel,

> On Mar 22, 2018, at 7:09 PM, Daniel Gustafsson <daniel@yesql.se> wrote:
>
> Unless there are objections I intend to commit that, but I wouldn’t mind an
> extra set of eyes on it.

So I don’t object on the patch itself (I have not reviewed but I have confidence
it’s okay).

However, we are close to committing a large project that involves essentially
a full HTML rewrite, so I would be hesitant to commit this right at this moment,
as it could be mooted in fairly short order and I don’t want your work to have
been for naught.

Jonathan