Skip to content

doc: Use neutral domains.#6471

Merged
traefiker merged 1 commit intotraefik:v2.2from
ldez:feat/neutral-domains
Mar 13, 2020
Merged

doc: Use neutral domains.#6471
traefiker merged 1 commit intotraefik:v2.2from
ldez:feat/neutral-domains

Conversation

@ldez
Copy link
Contributor

@ldez ldez commented Mar 11, 2020

What does this PR do?

Use only the reserved example second level domain names in the documentation.

https://tools.ietf.org/html/rfc2606#section-3

Motivation

Have a better documentation.

Fixes #6470

More

  • [ ] Added/updated tests
  • Added/updated documentation

Additional Notes

Copy link
Collaborator

@jbdoumenjou jbdoumenjou left a comment

Choose a reason for hiding this comment

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

LGTM

@ldez ldez added bot/light-review decreases the number of required LGTM from 3 to 1. status/3-needs-merge and removed status/2-needs-review labels Mar 13, 2020
@traefiker traefiker merged commit 5adf74e into traefik:v2.2 Mar 13, 2020
@traefiker traefiker removed bot/light-review decreases the number of required LGTM from 3 to 1. status/3-needs-merge labels Mar 13, 2020
@ldez ldez deleted the feat/neutral-domains branch March 15, 2020 23:48
@jsoref
Copy link
Contributor

jsoref commented Sep 3, 2024

Fwiw, this change corrupted a number of the examples.

The spirit of the change is absolutely desirable, but the examples had more problems (including scheese.com in places) which meant that a naive search and replace didn't achieve the desired results (using the RFC example.com domain).

But, ignoring the fact that a search and replace of bad content results in bad content, a search and replace of cert content that doesn't fix up the encoded form is problematic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants