Skip to content

Conversation

@r-birkner
Copy link

No description provided.

@kennetheversole
Copy link
Contributor

Can you please provide more context on the issue this was causing?

@r-birkner
Copy link
Author

We use this crate to create DNS records for the ACME-challenge. With this field, there would be a panic because the crate failed to deserialize the response as the field was missing (it has been deprecated, see here).

I have noticed that you have overhauled the codebase quite a bit. We are running off of a fork with the applied fixes. I can try switching back to the upstream and see whether it works now.

@Riceman2000
Copy link
Contributor

@kennetheversole
Here is the error message produced when DNS records are listed with version 0.13.0:

API request to list existing records failed with: Invalid(reqwest::Error { kind: Decode, source: Error("missing field `auto_added`", line: 1, column: 181) })

Thanks @r-birkner for putting this fix together.

@GoncaloGarcia
Copy link
Contributor

@r-birkner Could you rebase this please?

@ErcinDedeoglu
Copy link

?

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.

5 participants