Skip to content
This repository has been archived by the owner on Oct 4, 2023. It is now read-only.

[Bug]: TXT records renewal fails when there are old TXT records #3

Open
eze-kiel opened this issue Feb 10, 2023 · 0 comments
Open

[Bug]: TXT records renewal fails when there are old TXT records #3

eze-kiel opened this issue Feb 10, 2023 · 0 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@eze-kiel
Copy link
Contributor

Version

v1.1.0

What happened?

Here is what happens:

image

  1. The cert needs to be renewed, as new TXT records are available in Cloudflare.
  2. cfcr gets them, and then check if some TXT records are already present in OVH.
  3. As cfcr is supposed to be the only one that handles those TXT records, it thinks that it is the one that put them there, so there is no renewal to perform. It knows that because when the cert is renewed, it cleans the TXT records. So if there are TXT records remaining, it means that they are still useful.

Relevant log output

No response

Anything else?

To fix it, cfcr should compare the TXT records it got from Cloudflare to the one it got from OVH. Cloudflare is the source of truth, so:

  1. if they are the same, all good, just wait & pray.
  2. if they are different, replace the ones from OVH by the ones from Cloudflare.
@eze-kiel eze-kiel added bug Something isn't working good first issue Good for newcomers labels Feb 10, 2023
@eze-kiel eze-kiel self-assigned this Feb 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant