facebookarchive / libphenom

An eventing framework for building high performance and high scalability systems in C.
http://facebook.github.io/libphenom
Apache License 2.0
1.66k stars 362 forks source link

RFC: Remove libcares DNS support in favor of getdnsapi #57

Closed wez closed 10 years ago

wez commented 10 years ago

I want to remove libcares support because I'm not especially happy with either its API or how it is exposed up through libphenom.

The getdnsapi project proposes a promising modern API that is easier to use and supports features like DNSSEC.

This issue is tracking booting out libcares support and instead making it very easy to enable support for getdnsapi implementations. There is currently only one implementation, built on top of libunbound.

libunbound uses libevent under the covers, it feels a bit weird to have two eventing libraries running but it doesn't harm anything from a practical perspective.

I'm tracking this as a low-pri issue as it isn't a burning problem. I am sensitive to breaking existing integrations, so putting this out for feedback before I just go ahead and remove support for it.

wez commented 10 years ago

Removed. I'll defer the getdnsapi side of this for now, and revisit if/when there is more interest.

kev009 commented 10 years ago

The top level readme still refs cares. Definitely interested in getdnsapi at some point.