Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Loop redirection between two URLs #11080

Open
3 tasks done
yin1999 opened this issue May 7, 2024 · 3 comments
Open
3 tasks done

Loop redirection between two URLs #11080

yin1999 opened this issue May 7, 2024 · 3 comments
Labels
infra needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened.

Comments

@yin1999
Copy link
Member

yin1999 commented May 7, 2024

Summary

Got a loop redirection between the following URLs:

This may be related to mdn/content#33339.

URL

Reproduction steps

  1. Open browser with devtools opened in Network panel
  2. navigator to https://developer.mozilla.org/en-US/docs/Web/API/Window/performance
  3. See network logs

Note, clearing website data cannot resolve this problem, it seems to be a server-side problem.

Expected behavior

Won't get a loop redirection, and this page can be opened as normal.

Actual behavior

Got a loop redirection.

Device

Desktop

Browser

Chrome

Browser version

Stable

Operating system

Windows

Screenshot

image

Anything else?

No response

Validations

@github-actions github-actions bot added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label May 7, 2024
@bsmth
Copy link
Member

bsmth commented May 7, 2024

Already discussed on Discord, but adding a note to say I can't reproduce this, it may have been caches being invalidated during deploy.

@bsmth
Copy link
Member

bsmth commented May 8, 2024

For some more context, the problem appears to be from here:

mdn/content@dfb8c72#diff-afe010c4707d10d488b1e1c050b3298da03267dec0abca46d6d444b2ae8774a5L10890

We had a situation with these changes in the redirects file:

- /en-US/docs/Web/API/Window/performance /en-US/docs/Web/API/performance_property
+ /en-US/docs/Web/API/performance_property /en-US/docs/Web/API/Window/performance

And the old & new redirects were both active at the same time.

@bsmth bsmth added the infra label May 8, 2024
@caugner
Copy link
Contributor

caugner commented May 14, 2024

I cannot reproduce the issue, and indeed this indicates a temporary caching issue. We tell browsers to cache permanent redirects for moved pages for 30 days (which we could reduce), and our CDN only caches them for 30 minutes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened.
Projects
None yet
Development

No branches or pull requests

3 participants