-
Notifications
You must be signed in to change notification settings - Fork 549
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Problems updating DNS record #97
Comments
I was able to update the record when hitting the API directly:
|
Ah, there is indeed a very stupid bug here. I think when it was originally written an assumption was made that anything but the label name would be changed, so I'm not sure why the TTL isn't being updated, though. Maybe an artefact of the entire record not being updated? |
Hopefully the PR fixes this for you. Are you able to test it before merging? |
@jamesog I'd be happy to test, but not sure how.. I tried checking out Not sure if this is an issue with how I did things or if there's still an issue with the library. Maybe you could add some debug output so I'm sure that I'm actually running your code? 😄 Output
Web Console |
@jamesog I added some extra debugging logs, and noticed I was trying to update the record to |
Cool, thanks! |
@jamesog Awesome, thanks for the quick response on this! 👍 |
Welcome :-) |
Hey there, I'm looking into hashicorp/terraform#7690, and trying to determine if the issue is in the terraform code or in the Cloudflare library.
I'm having issues updating a record using
cloudflare-go
directly (I can create it ok). Wondering if someone can point out if I'm doing something incorrectly, or if this is actually a bug in the library.I'm pretty new to golang, so not surprised if I'm doing something incorrectly here.
cloudflare.go
Output
Cloudflare Web Console
Expected result
Name=cfapi-modified
Content=192.168.10.10
TTL=120s
The text was updated successfully, but these errors were encountered: