You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It seems like this crate currently only supports the most common DNS record types:
// in cloudflare/src/endpoints/dns.rspubenumDnsContent{A{content:Ipv4Addr},AAAA{content:Ipv6Addr},CNAME{content:String},NS{content:String},MX{content:String,priority:u16},TXT{content:String},}
However, the Cloudflare API lists several additional record types that are currently not implemented, including CAA, CERT, DNSKEY and more.
From what I can tell, adding any of these records to a zone causes the ListDnsRecords query to fail with an unknown variant error during deserialization, effectively making the ListDnsRecords query unavailable for such a zone (example below).
I'm running into this issue on my own zones, since I have a CAA record set on them.
I have a fork that fixes this for CAA records, but I'd be interested in seeing support for these record types in this main crate.
Adding simple enum variants for these records with the content field should be sufficient to allow the ListDnsRecords request to succeed (the API also returns a structured data field for some record types, but I don't think this is necessary for basic support).
If support for these record types is desired, I'd be more than happy to make the required changes and open a PR!
Steps to reproduce:
Create a zone and create a CAA record through the Web Interface (such as 0 issue letsencrypt.org).
Attempt to list all records in that zone using the cloudflare-examples utility like so:
cargo run -p cloudflare-examples -- --auth-token <redacted> dns <zone-id>
The client will return the following error:
Error: error decoding response body: unknown variant `CAA`, expected one of `A`, `AAAA`, `CNAME`, `NS`, `MX`, `TXT`, `SRV` at line 1 column 8705
The text was updated successfully, but these errors were encountered:
maxhoesel
changed the title
DNS: Suuport for additional record types
DNS: Suport for additional record types
Sep 14, 2023
maxhoesel
changed the title
DNS: Suport for additional record types
DNS: Support for additional record types
Sep 14, 2023
It seems like this crate currently only supports the most common DNS record types:
However, the Cloudflare API lists several additional record types that are currently not implemented, including
CAA
,CERT
,DNSKEY
and more.From what I can tell, adding any of these records to a zone causes the
ListDnsRecords
query to fail with anunknown variant
error during deserialization, effectively making theListDnsRecords
query unavailable for such a zone (example below).I'm running into this issue on my own zones, since I have a
CAA
record set on them.I have a fork that fixes this for
CAA
records, but I'd be interested in seeing support for these record types in this main crate.Adding simple enum variants for these records with the
content
field should be sufficient to allow theListDnsRecords
request to succeed (the API also returns a structureddata
field for some record types, but I don't think this is necessary for basic support).If support for these record types is desired, I'd be more than happy to make the required changes and open a PR!
Steps to reproduce:
0 issue letsencrypt.org
).cloudflare-examples
utility like so:The text was updated successfully, but these errors were encountered: