cURL
Haxx ad
libcurl

curl's project page on SourceForge.net

Sponsors:
Haxx

cURL > Mailing List > Monthly Index > Single Mail

curl-library Archives

Re: [PATCH]: allow clients to set the name servers

From: Jason Glasgow <jglasgow_at_google.com>
Date: Sun, 6 Nov 2011 00:03:24 -0400

On Thu, Sep 29, 2011 at 11:43 PM, Jason Glasgow <jglasgow_at_google.com> wrote:
> Daniel & Ben,
> Thanks for the feedback.  Let me try to address the issues you raise and
> I'll send an updated patch.
> -Jason
>
> On Thu, Sep 29, 2011 at 2:14 AM, Daniel Stenberg <daniel_at_haxx.se> wrote:
>>
>> On Wed, 28 Sep 2011, Ben Greear wrote:
>>
>>> Last time I posted a similar patch, I think it was requested that the
>>> option name be more generic.  I ended up with CURLOPT_DNS_SERVERS, just in
>>> case some day it supports more than just c-ares.
>>
>> Good point!
>>
>> I would also prefer to have the function return CURLE_NOT_BUILT_IN if
>> libcurl wasn't built to support the option (rather than unknown option).
>>
>> --
>>
>>  / daniel.haxx.se

It took me awhile to get back to this patch. Here is a second try
that I think addresses all the concerns short of testing. If this
looks acceptable, I will try to figure out how to properly test this
in the existing test framework.
Received on 2011-11-06

These mail archives are generated by hypermail.

donate! Page updated November 12, 2010.
web site info

File upload with ASP.NET