[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