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

TTL? #1

Closed
bclaymiles opened this issue Aug 5, 2019 · 2 comments
Closed

TTL? #1

bclaymiles opened this issue Aug 5, 2019 · 2 comments

Comments

@bclaymiles
Copy link

bclaymiles commented Aug 5, 2019

Hey there, just tried this out, it's a neat idea - but if I set a TTL on the records, the server app dies with 'ValueError: too many values to unpack'.

Without a TTL, one is forced to wait on djb's default TTL's, which are much too long to respond to a dynamic change.

Is there a workaround -- a way to force a lower TTL?

@mk-fg
Copy link
Owner

mk-fg commented Aug 6, 2019

Hi,

It's just a trivial bug, forgot to put a limit on unpacked line parts apparently, didn't think to add line to test that case and don't use ttls with these zones myself.
Fixed in a31fdda, should work with any number of extra parameters, including TTL, as intended.

Thanks for reporting.

Note that I wrote this a while ago in python2, which goes EOL later this year, though probably will still be around in a deprecated state for a decade, if not more.
Might update it to python3 at some point (should be rather easy, and netaddr is built-in as ipaddress module there), but not sure if maybe will replace remaining two tinydns setups I have with nsd instead.

@mk-fg mk-fg closed this as completed Aug 6, 2019
@bclaymiles
Copy link
Author

Thanks so much for jumping into this! I'll give it a whirl soon. cheers!

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

2 participants