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

Print error details when an IssuanceError is thrown #9804

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

TheEnbyperor
Copy link
Contributor

When a CA fails to issue a certificate after finalisation Certbot currently prints the following unhelpful message:

An unexpected error occurred:
acme.errors.IssuanceError

This PR makes Certbot print the ACME error object from the order, as such

An unexpected error occurred:
CAA error :: Invalid CAA: CAA prohibits issuance

Pull Request Checklist

  • The Certbot team has recently expressed interest in reviewing a PR for this. If not, this PR may be closed due our limited resources and need to prioritize how we spend them.
  • 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.

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.

None yet

1 participant