Skip to content
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

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
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers todo 🗒️

Comments

@DesertCookie
Copy link

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?

@siennathesane siennathesane self-assigned this Jan 19, 2020
@siennathesane siennathesane added bug Something isn't working documentation Improvements or additions to documentation good first issue Good for newcomers labels Jan 19, 2020
@siennathesane
Copy link
Owner

@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.

@siennathesane siennathesane added todo 🗒️ enhancement New feature or request labels Jan 19, 2020
@DesertCookie
Copy link
Author

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.

@siennathesane
Copy link
Owner

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

@siennathesane
Copy link
Owner

@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?

siennathesane 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 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 join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers todo 🗒️
Projects
None yet
Development

No branches or pull requests

2 participants