[Asterisk-code-review] Add a negative DNS cache (asterisk[master])

[Joshua Colp]

Joshua Colp has posted comments on this change.

Change subject: Add a negative DNS cache
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/1710/2/main/dns_core.c
File main/dns_core.c:

Line 539: 			ast_dns_cache_add_or_update(query->name);
I've not a huge fan of overriding the TTL as provided in the DNS resolution result, it may be lower than what is configured telling us to try back sooner.


-- 
To view, visit https://gerrit.asterisk.org/1710
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Icb9f8c7ec15eedd4701461bf865e8036a5732388
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-HasComments: Yes