search

monostream / cert-manager-linode

a cert-manager webhook adapter for linode
MIT License
22 stars 9 forks source link

Strip trailing dot for new domain record #2

Closed jdhollander closed 2 years ago

jdhollander commented 2 years ago

Fixes issue #1 (Error: [400] [name] Name contains invalid characters).

I have tested this on my own setup:

  1. Created and pushed new docker image with my changes.
  2. Replaced docker image with my own for the cert-manager-linode deployment on my cluster.
  3. Issued a new certificate with no problems, watched as the _acme-challenge record was succesfully created.