Skip to content

Only use *.example and example.com as example domains #41428

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Only use *.example and example.com as example domains #41428

wants to merge 1 commit into from
+78 −75

Conversation

Josh-Cena
Copy link
Member

Remove a lot of example domains that may happen to be valid, fixing #39865. It is impossible to find all such examples, but I found a good bunch.
@Josh-Cena Josh-Cena requested review from a team as code owners October 7, 2025 21:16
@Josh-Cena Josh-Cena requested review from chrisdavidmills, hamishwillee, rebloor and wbamberg and removed request for a team October 7, 2025 21:16
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs Content:WebExt WebExtensions docs Content:WebAPI Web API docs Content:JS JavaScript docs Content:Learn Learning area docs Content:HTTP HTTP docs Content:Glossary Glossary entries Content:Security Security docs Content:PWA Progressive Web Apps content labels Oct 7, 2025
@github-actions github-actions bot added the size/m [PR only] 51-500 LoC changed label Oct 7, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

Preview URLs (27 pages)
Flaws (3)

Note! 26 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/HTTP/Guides/Proxy_servers_and_tunneling/Proxy_Auto-Configuration_PAC_file
Title: Proxy Auto-Configuration (PAC) file
Flaw count: 3

  • unknown:
    • No generic content config found
    • no blog root
    • no blog root
External URLs (1)

URL: /en-US/docs/Web/API/Storage_Access_API
Title: Storage Access API
bsmth
bsmth reviewed Oct 8, 2025
View reviewed changes
files/en-us/web/http/reference/headers/access-control-allow-headers/index.md
Access-Control-Request-Method: GET
Access-Control-Request-Headers: content-type,x-requested-with
Origin: https://foo.bar.org
Origin: https://example.com
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want to keep a subdomain here and the one below?

Suggested change
Origin: https://example.com
Origin: https://subdomain.example.com
rebloor
rebloor reviewed Oct 8, 2025
View reviewed changes
files/en-us/mozilla/add-ons/webextensions/api/cookies/index.md
Comment on lines +42 to +43
1. You visit `a-shopping-site.example`, which uses `ad-tracker.example` to deliver its adverts on the web. `ad-tracker.example` sets a cookie associated with the `ad-tracker.example` domain. While you are on `a-shopping-site.example`, `ad-tracker.example` receives information about the products you browse.
2. You now visit `a-news-site.example` that uses `ad-tracker.example` to deliver adverts. `ad-tracker.example` read its cookie and use the information collected from `a-shopping-site.example` to decide which adverts to display to you.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dotproto any thoughts on this? Does it make the examples to unrealistic?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs Content:Glossary Glossary entries Content:HTTP HTTP docs Content:JS JavaScript docs Content:Learn Learning area docs Content:PWA Progressive Web Apps content Content:Security Security docs Content:WebAPI Web API docs Content:WebExt WebExtensions docs size/m [PR only] 51-500 LoC changed

3 participants

@Josh-Cena @rebloor @bsmth