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

Change insecure links #11

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

HerrSubset
Copy link

Hi there 👋

I noticed an issue when clicking the "LabelSelector" link in the README. My browser (Firefox) notified me that the certificate for that site was invalid.
I guess this is because the Kubernetes docs only list v1.19 and later, and the link points to the v1.18 docs?

In any case, I changed the links to point to a more recent version of the docs, but please check if that link is ok.
I also took the liberty of moving all links to the bottom of the page. Then the LabelSelector link can be de-duplicated, and the file is a bit nicer to read.

Pieter-Jan Smets added 2 commits March 1, 2022 17:50
- one link can be de-duplicated
- text becomes easier to read
The README contained a link pointing to the docs of Kubernetes v1.18.
The docs for that version are no longer online, and when visiting the
link your browser shows a message about an invalid certificate.

In this commit, I changed the file to point to a newer version of the docs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant