Skip to content

gh-93573: Replace wrong example domains in configparser doc#93574

Merged
JulienPalard merged 3 commits into
python:mainfrom
sblondon:remove-bytebong.com
Feb 16, 2023
Merged

gh-93573: Replace wrong example domains in configparser doc#93574
JulienPalard merged 3 commits into
python:mainfrom
sblondon:remove-bytebong.com

Conversation

@sblondon

@sblondon sblondon commented Jun 7, 2022

Copy link
Copy Markdown
Contributor

This minor PR replaces bytebong.com to python.org in configparser example in order to use a domain the PSF controls.

topsecret.server.com is obviously an example and I don't think bitbucket.org is a risk currently so they are not modified.

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Jun 7, 2022
@arhadthedev

Copy link
Copy Markdown
Member

topsecret.server.com

Maybe, topsecret.example.com would be better?

@tiran

tiran commented Jun 7, 2022

Copy link
Copy Markdown
Member

Documentation should use reserved TLDs from https://datatracker.ietf.org/doc/html/rfc2606, e.g. topsecret.python.example.

topsecret.server.com domain is not controled by PSF. It's replaced by invalid topsecret.server.example domain. It follows RFC 2606, which advise .example as TLD for documentation.
@sblondon

sblondon commented Jun 7, 2022

Copy link
Copy Markdown
Contributor Author

I replaced topsecret.server.com by topsecret.server.example so the difference between topsecret.server.example and python.org assertions are more obvious.

I don't think news are needed.

@AA-Turner AA-Turner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If we're changing to .example, it might also be worth changing the bitbucket.org ones? I know the domain is currently active, but extra safety doesn't hurt.

A

@sblondon

sblondon commented Jun 7, 2022

Copy link
Copy Markdown
Contributor Author

bitbucket.org domain is now replaced by forge.example.

@AA-Turner AA-Turner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

A

@AA-Turner AA-Turner added needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels Jun 7, 2022
@sblondon sblondon changed the title gh-93573: Replace bytebong.com by python.org gh-93573: Replace unused domains to invalid ones, and bytebong.com to python.org Jun 14, 2022
@merwok

merwok commented Jun 14, 2022

Copy link
Copy Markdown
Member

IMHO server.example is not as easily understood as a domain name as server.example.com

@sblondon

Copy link
Copy Markdown
Contributor Author

IMHO server.example is not as easily understood as a domain name as server.example.com

RFC2606 reserves also example.{com,net,org}.
@tiran what do you think about it?

@merwok merwok changed the title gh-93573: Replace unused domains to invalid ones, and bytebong.com to python.org gh-93573: Replace wrong example domains in configparser doc Jun 28, 2022
@sblondon

Copy link
Copy Markdown
Contributor Author

The bytebong.com domain is now available to everyone:

$ whois bytebong.com
No match for domain "BYTEBONG.COM".
>>> Last update of whois database: 2022-11-21T12:47:09Z <<<
...

Should I ping a core reviewer for the merge?

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @sblondon for the PR, and @JulienPalard for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-101953 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Feb 16, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 16, 2023
…thonGH-93574)

* Replace bitbucket.org domain by forge.example
* Update example to python.org
* Use explicitly invalid domain

topsecret.server.com domain is not controled by PSF. It's replaced by invalid topsecret.server.example domain. It follows RFC 2606, which advise .example as TLD for documentation.
(cherry picked from commit 924a3bf)

Co-authored-by: sblondon <stephane.blondon@gmail.com>
@bedevere-bot

Copy link
Copy Markdown

GH-101954 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Feb 16, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 16, 2023
…thonGH-93574)

* Replace bitbucket.org domain by forge.example
* Update example to python.org
* Use explicitly invalid domain

topsecret.server.com domain is not controled by PSF. It's replaced by invalid topsecret.server.example domain. It follows RFC 2606, which advise .example as TLD for documentation.
(cherry picked from commit 924a3bf)

Co-authored-by: sblondon <stephane.blondon@gmail.com>
miss-islington added a commit that referenced this pull request Feb 16, 2023
* Replace bitbucket.org domain by forge.example
* Update example to python.org
* Use explicitly invalid domain

topsecret.server.com domain is not controled by PSF. It's replaced by invalid topsecret.server.example domain. It follows RFC 2606, which advise .example as TLD for documentation.
(cherry picked from commit 924a3bf)

Co-authored-by: sblondon <stephane.blondon@gmail.com>
miss-islington added a commit that referenced this pull request Feb 16, 2023
* Replace bitbucket.org domain by forge.example
* Update example to python.org
* Use explicitly invalid domain

topsecret.server.com domain is not controled by PSF. It's replaced by invalid topsecret.server.example domain. It follows RFC 2606, which advise .example as TLD for documentation.
(cherry picked from commit 924a3bf)

Co-authored-by: sblondon <stephane.blondon@gmail.com>
@sblondon sblondon deleted the remove-bytebong.com branch February 16, 2023 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants