Skip to content
This repository has been archived by the owner. It is now read-only.

Tries to update AAAA record with IPv4 #12

Open
DesertCookie opened this issue Jan 18, 2020 · 5 comments
Open

Tries to update AAAA record with IPv4 #12

DesertCookie opened this issue Jan 18, 2020 · 5 comments

Comments

@DesertCookie
Copy link

@DesertCookie DesertCookie commented Jan 18, 2020

Event though I have set type to AAAA in my config file, cloudflare-sync attempts to send an IPv4 to CloudFlare (which my PC does not have), thus resulting in a failure to update.

Are IPv6 records even supported by cloudflare-sync?

@mxplusb
Copy link
Owner

@mxplusb mxplusb commented Jan 19, 2020

@DesertCookie currently they are not supported since I didn't use it at the time I wrote it (and still don't), but I could add support for it if it's a blocker for usage.

Sorry, it was a very basic MVP I wrote, mostly for myself, but if you have a need, I can totally do it, might take a bit though.

@DesertCookie
Copy link
Author

@DesertCookie DesertCookie commented Jan 20, 2020

Support for AAAA records would indeed be great. Don't hustle yourself though - maybe I'm quicker writing a program myself. But since IPv6 is the default since 2012 an the industry is slowly but surely transitioning, I think future-proofing your application is always a good idea.

@mxplusb
Copy link
Owner

@mxplusb mxplusb commented Jan 22, 2020

I'll take a look and see how hard it would be to implement. :)

@mxplusb
Copy link
Owner

@mxplusb mxplusb commented Jan 28, 2020

@DesertCookie I updated it to support AAAA records, but I can't test it because my ISP doesn't provide IPv6 addresses without some hassle. Can you test the latest commit and provide some feedback?

mxplusb added a commit that referenced this issue Jan 28, 2020
asked for feedback with #12

Signed-off-by: Mike Lloyd <mike@reboot3times.org>
@DesertCookie
Copy link
Author

@DesertCookie DesertCookie commented Feb 2, 2020

After messing around for a while and solving several problems that caused errors to be thrown, this is the one I cannot get past: cannot unmarshal production configuration file.
My configuration file looks like this (reducted):

{
  "zone_id": "...",
  "zone_name": "domain.ext",
  "api_token": "...",
  "frequency": "30",
  "records": [
    {
      "type": "AAAA",
      "name": "sub.domain.ext",
      "proxied": true,
      "ttl": 120,
      "priority": 1
    }
  ]
}
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.