azurerm_dns_x_record: ignore case on target_resource_id

[magodo]

In some case (at least for the cdn endpoint), the ID of target resource
read from API (which uses the API model of the target resource) doesn't match
the target_resource_id if it is imported via Terraform (which actually
uses the API model of the azurerm_dns_x_record) in casing.

Fixes: #8191

[tombuildsstuff]

@magodo is there an Swagger bug to track this API bug?

ARM API's are supposed to be following Postel's law, where they are liberal in what they accept (eg uri's could be case insensitive) but conservative in what they send (eg case sensitive) - so this is a bug in the API which needs to be fixed

Whilst applying a DiffSuppress does work around this - it's something that we should avoid unless absolutely necessary, so it'd be worth raising the API bug here

[dossy]

@tombuildsstuff, @magodo - I'm not familiar enough with TF's code (yet), but I feel like the issue is actually with azurerm_cdn_endpoint's id output not properly casing resourcegroups as resourceGroups.

[magodo]

Close in favor of #8237, which modify the resource id of cdn endpoint.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!