paultyng / terraform-provider-unifi

Terraform provider for Unifi :satellite:
https://registry.terraform.io/providers/paultyng/unifi/latest/docs
Mozilla Public License 2.0
510 stars 90 forks source link

Fix missing IP on unifi_user bug #430

Open ProfMoo opened 10 months ago

ProfMoo commented 10 months ago

Hi! Thanks for making this TF provider 😄.

I was trying to use this provider to get an IP address based on a known MAC address, but was getting null results back from the TF data resource. I manually checked the REST endpoint to make sure the data was available, which it was.

After some investigation, I believe an unintentional bug was introduced in this PR which overwrites the IP even if there is no local DNS record found. I have added a conditional that only overwrites the IP if there is a local DNS record instead.

I have tested this locally and it works in my scenario now 😄