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

Does geodns support cname chain? #63

Open
li45 opened this issue Apr 29, 2015 · 5 comments
Open

Does geodns support cname chain? #63

li45 opened this issue Apr 29, 2015 · 5 comments

Comments

@li45
Copy link

li45 commented Apr 29, 2015

No description provided.

@abh
Copy link
Owner

abh commented Apr 29, 2015 via email

@li45
Copy link
Author

li45 commented Apr 30, 2015

my test.com.json:

    "bar" : {
        "a":[["192.168.1.2",100]]
    },
    "www":  {
        "cname": "bar.test.com."
    }

then run dig @127.0.0.1 www.test.com
response:

;; QUESTION SECTION:
;www.test.com.          IN  A

;; ANSWER SECTION:
www.test.com.   60  IN  CNAME   bar.test.com.

no A recode in here.

Expect response:

;; QUESTION SECTION:
;www.test.com.          IN  A

;; ANSWER SECTION:
www.test.com.   60  IN  CNAME   bar.test.com.
bar.test.com.    60 IN  A   192.168.1.2

@abh
Copy link
Owner

abh commented Apr 30, 2015

Ah, yes — I believe it works because the resolver will just query again for the other name, but obviously that could be optimized!

There’s an “alias” (I think it’s called) record type that “resolves” the CNAME internally.

I should fix the cname chain thing though.

@semihalev
Copy link

abh, for glue records, check my commit semihalev@fc96682, resolve internally for CNAME, MX, NS, but need small boolean (msg.CheckGlue) in dns lib.

@praveenkumar2608
Copy link

Ah, yes — I believe it works because the resolver will just query again for the other name, but obviously that could be optimized!

There’s an “alias” (I think it’s called) record type that “resolves” the CNAME internally.

I should fix the cname chain thing though.

@abh @semihalev Is this fixed. Facing same issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants