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

Add wildcard example #9164

Merged
merged 2 commits into from
Jan 18, 2022
Merged

Add wildcard example #9164

merged 2 commits into from
Jan 18, 2022

Conversation

kevgrig
Copy link
Contributor

@kevgrig kevgrig commented Jan 9, 2022

Pull Request Checklist

  • If the change being made is to a distributed component, edit the master section of certbot/CHANGELOG.md to include a description of the change being made.
  • Add or update any documentation as needed to support the changes in this PR.
  • Include your name in AUTHORS.md if you like.

--dns-digitalocean \\
--dns-digitalocean-credentials ~/.secrets/certbot/digitalocean.ini \\
-d example.com \\
-d *.example.com
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't it be enclosed in simple quotes to avoid the shell expansion on wildcard characters?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I recently ran it successfully without any quoting but it doesn't hurt to add, so I'll add that...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

New commit pushed

@bmw
Copy link
Member

bmw commented Jan 18, 2022

@adferrand, do you have the time & interest to rereview this or should one of us take it?

@bmw bmw added the more-info label Jan 18, 2022
@adferrand
Copy link
Collaborator

No that is fine for me, thanks for the reminder !

@adferrand adferrand merged commit afc5be5 into certbot:master Jan 18, 2022
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.

3 participants