ansible-collections / community.general

Ansible Community General Collection
https://galaxy.ansible.com/ui/repo/published/community/general/
GNU General Public License v3.0
798 stars 1.47k forks source link

cloudflare_dns does not support CAA CERT, DNSKEY, LOC, NAPTR, PTR, SMIMEA, URI record types #826

Open pxeger opened 3 years ago

pxeger commented 3 years ago
SUMMARY

Extension/revival of ansible/ansible#43803. The cloudflare_dns module doesn't support all the newest RR types:

ISSUE TYPE
COMPONENT NAME

cloudflare_dns

ADDITIONAL INFORMATION

Exactly how the current records work. Some new parameters would probably need to be added to support the various record fields.

ansibullbot commented 3 years ago

Files identified in the description:

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

ansibullbot commented 3 years ago

cc @mgruener @nerzhul click here for bot help

pxeger commented 3 years ago

Looking at the source code, it's pretty messy and could do with some cleaning up in general. Can I do a PR?

felixfontein commented 3 years ago

@pxeger I guess so, though I'd wait a bit to see whether the maintainers react. Anyway, if you want to refactor it, please make separate PRs for refactoring and adding features. That makes it easier to review.

aminvakil commented 3 years ago

+label waiting_on_contributor

ansibullbot commented 1 year ago

Files identified in the description:

If these files are incorrect, please update the component name section of the description or use the !component bot command.

click here for bot help

felixfontein commented 7 months ago

CAA support has been added in #7399.