reactphp/dns

DNS cache should properly limit TTL

clue opened this issue · 1 comments

clue commented

This component and its cache support works reasonably well currently.

However, this implementation involves a lot of code because it was actually written before the underlying cache component supports TTL values. Once the underlying cache supports proper TTL values for its cache values via reactphp/cache#11, we should update this component to take advantage of this.

In particular, we should carefully follow the spec to apply proper TTL values, such as when a single query returns two records with varying TTLs.

Opening this ticket as a reminder – the implementation actually depends on reactphp/cache#11