Fix creation of srv entries #5167

[martin-schlossarek]

SUMMARY

Fixes #5167

This PR removes the splicing of the record entry which resulted in malformed api calls. record must be the name of the subdomain or "@"

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

cloudflare_dns

ADDITIONAL INFORMATION

See: #5167 (comment)

[ansibullbot]

cc @mgruener
click here for bot help

[felixfontein]

Pending discussion in #5167.

[felixfontein]

Please note that in #5461 the collection repository was restructured to remove the directory tree in plugins/modules/, and the corresponding tree in tests/unit/plugins/modules/. Your PR modifies files in this directory structure, and unfortunately now has some conflicts, potentially because of this. Please rebase with the current main branch to remove the conflicts.

[martin-schlossarek]

Sorry, I didn't have time to look at the problem further. In the meantime my use case to be able to set SRV entries has also become obsolete.

I withdraw the pull request, if that's ok?

Nevertheless, I think it should be mentioned in the documentation that SRV entries must always have a FQDN specified. For other entries this is not the case.

[github-actions]

Docs Build 📝

This PR is closed and any previously published docsite has been unpublished.

[felixfontein]

Nevertheless, I think it should be mentioned in the documentation that SRV entries must always have a FQDN specified. For other entries this is not the case.

Do you want to create a PR for that? :)