amphp / dns

Async DNS resolution for PHP based on Amp.
https://amphp.org/dns
MIT License
157 stars 32 forks source link

Invalid promise resolution for cached failures #26

Closed kelunik closed 8 years ago

kelunik commented 8 years ago

We currently cache in case of no results for a maximum of 5 minutes. The first invocation results in a ResolutionException (should actually be NoRecordException, see #25). All later invocations result in an successfully resolved promise yielding an empty array. Instead there should be the same outcome as for non-cached DNS requests (the same exception).