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

fix .com and .org domain in documentation #10635

Merged
merged 1 commit into from
May 27, 2024
Merged

Conversation

rptaylor
Copy link

What does this PR do?

fix documentation error

Motivation

Based on https://github.com/traefik/traefik/blob/v2.11/docs/content/https/acme.md, which says

* A certificate for the domains `example.com` (main) and `*.example.org` (SAN)
  is requested:

It looks like the main domain should be example.com and the SAN should be example.org, but the included YAML files did not show that.

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

Please check if this is correct.

@mloiseleur
Copy link
Contributor

This fix looks correct to me.

Copy link
Contributor

@nmengin nmengin left a comment

Choose a reason for hiding this comment

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

Hello @rptaylor,

Thank you for your contribution 👍
LGTM

@nmengin nmengin closed this May 24, 2024
@nmengin nmengin reopened this May 24, 2024
@nmengin
Copy link
Contributor

nmengin commented May 24, 2024

FYI I closed/re-opened the PR to launch the CI.

@nmengin nmengin removed their assignment May 24, 2024
Copy link
Member

@juliens juliens left a comment

Choose a reason for hiding this comment

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

LGTM

@traefiker
Copy link
Contributor

🚫 checks status: PR status: failure

@traefiker traefiker merged commit 4406c33 into traefik:v2.11 May 27, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants