[sr-dev] http_client: "curlcon" name

Olle E. Johansson oej at edvina.net
Wed Feb 3 12:05:55 CET 2016


> On 03 Feb 2016, at 11:53, Daniel-Constantin Mierla <miconda at gmail.com> wrote:
> 
> 
> 
> On 03/02/16 11:33, Olle E. Johansson wrote:
>>> On 03 Feb 2016, at 11:10, Daniel-Constantin Mierla <miconda at gmail.com> wrote:
>>> 
>>> I would try to keep http prefix for all the functions that do not
>>> overlap. I expect the functions from utils modules will be moved to one
>>> of the new http client modules in the future, given the better framework
>>> provided by these ones.
>> I already have a copy of that function, which was the base I started from.
>> I asked about deleting the function in utils on the mailing list earlier, but Juha
>> wasn’t positive to that idea so I let it go and changed the name in the curl
>> module.
> 
> Once we will get to the new consolidated http client module and has gone
> through some production battles, it will make no sense to keep the ones
> in utils. But for now the situation is a bit floating to get rid of the
> old ones, which are rather stable.
> 
Agree. Let’s clean up in version 5.

/O
> Cheers,
> Daniel
> 
>>> For the ones that overlap and don't provide (or target) the same
>>> functionality, then maybe something new can be found.
>>> 
>>> For the ones that overlap in functionality and target, until the old
>>> ones get merged, then we can use some other prefix, but I think http
>>> should be full in there, maybe http_curl_ or httpc_ -- again, in the
>>> plan should be merging the functions from utils into these ones in the
>>> near future, ending up in having full set of cunctions prefixed with http_.
>> I would like that.
>> 
>> /O
>>> Cheers,
>>> Daniel
>>> 
>>> On 03/02/16 10:56, Olle E. Johansson wrote:
>>>> I am about to rename functions with the “curl” prefix as well as the connection defintions.
>>>> 
>>>> Since the old function in utils use “http” as a prefix we need something else.
>>>> 
>>>> “http_client” seems too long
>>>> 
>>>> “hc” and “hac” for sync and async functions may work. “httpcon” instead of “curlcon”.
>>>> 
>>>> Any better ideas?
>>>> 
>>>> /O
>>>> _______________________________________________
>>>> sr-dev mailing list
>>>> sr-dev at lists.sip-router.org
>>>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
>>> -- 
>>> Daniel-Constantin Mierla
>>> http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
>>> Book: SIP Routing With Kamailio - http://www.asipto.com
>>> http://miconda.eu
>>> 
>>> 
>>> _______________________________________________
>>> sr-dev mailing list
>>> sr-dev at lists.sip-router.org
>>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
> 
> -- 
> Daniel-Constantin Mierla
> http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
> Book: SIP Routing With Kamailio - http://www.asipto.com
> http://miconda.eu
> 




More information about the sr-dev mailing list