[sr-dev] Patches for siptrace

[Daniel-Constantin Mierla]

Hello Alexandr,

indeed, this does not interfere with Homer project at all, just some 
adjustments to the existing siptrace: option to mirror traffic only (old 
version was always writing to db) and option to add extra headers when 
mirroring and read them when receiving mirroring traffic.

Cheers,
Daniel

On 8/15/11 1:37 AM, Alexandr Dubovikov wrote:
> Hello,
>
> The idea is good, but I am not sure it's correct to modify the 
> original SIP message, adding new SIP headers.
> As minimum you don't have any controlling, which headers are present 
> or must be present.
>
> Anyway, this solution can be used in some cases there is only Kamailo 
> present.
>
> Wbr,
> Alexandr
>
> P.S. it doesn't interrupt Homer implementation, it's just another way 
> of mirroring SIP traffic.  :-)
>
> P.P.S. http://homer.googlecode.com/files/Homerv2.pdf
>
>
> 8/12/2011 3:30 PM, Daniel-Constantin Mierla wrote:
>> Hello,
>>
>> let's keep the list on cc so other devs can participate and have idea 
>> of what is going on ...
>>
>> On 8/11/11 9:26 AM, Tristan Bruns wrote:
>>> Hello Daniel,
>>>
>>> I tried using the message parser and textops to add the headers to 
>>> the message but failed, probably because I did not manage to create 
>>> a sip_msg struct that append_hf would update. (It worked when I used 
>>> an existing sip_msg struct, [e.g. the one from sip_trace()] but not 
>>> all trace-functions had access to a sip_msg [e.g. trace_sl_*])
>>
>> Indeed, the functions for storing the message don't have the sip_msg 
>> struct available. I was referring to parsing of received messages 
>> (the mirrored traffic from other kamailios), where would be safer to 
>> use the parser -- if there will be another proxy in between mirroring 
>> proxy and storage proxy, it can add headers in between the 
>> X-Siptrace-..., breaking the read.
>>
>> Anyhow, that is very unlikely -- for the I will go with your patches 
>> and update afterwards, when it is more time for it.
>>
>> Cheers,
>> Daniel
>>
>>
>>
>> _______________________________________________
>> sr-dev mailing list
>> sr-dev at lists.sip-router.org
>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
>
>
>
> _______________________________________________
> 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://www.asipto.com
Kamailio Advanced Training, Oct 10-13, Berlin: http://asipto.com/u/kat
http://linkedin.com/in/miconda -- http://twitter.com/miconda

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20110817/ccd3622d/attachment.htm>