[Serdev] Re: rtpproxy patches

[Maxim Sobolev]

Daniel Pocock wrote:
> 
> 
> Jan Janak wrote:
> 
>> Hi Daniel,
>>
>> I forwarded your message to Maxim's new address <sobomax at sippysoft.com>
>> and also to serdev at lists.iptel.org
>>
>>  
>>
> Thanks Jan, I noticed the old one bounced.
> 
> The old one is still listed in the AUTHORS file in the tgz for 0.3-1 in 
> the Debian source package.

Thanks, I should have fixed them all already.

> 
> 
>>  Jan.
>>
>> Daniel Pocock wrote:
>>  
>>
>>> Hi Maxim,
>>>
>>> I've been working with your rtpproxy code.  I am currently running 0.2
>>> and I have reviewed the 0.3-1 source as well.
>>>
>>> I've made the following changes which apply to both versions, would you
>>> be interested in using these?
>>>
>>> a) rtpp_session.ttl is now an array
>>> rtpp_session.ttl[0] represents the ttl for caller
>>> rtpp_session.ttl[1] represents the ttl for callee
>>> If either ttl reaches 0, session is dropped
 >>> Change (a) is something that some people might want to vary - some 
users
 >>> may want to revert to old behaviour.  I can add a command line option
 >>> for reverting to the old behaviour.

Can you explain the purpose of this change? To me it seems like it will 
be a problem when either party puts his session on hold. In any case it 
should not be a default behaviour to not break POLA. And I am not sure 
if new command line option is really way to go - it's better to allow 
enabling it via U/L option (for example you can set ttl[1] to -1 to 
indicate that the old behaviour is expected).

>>> b) the command line parameter `-m unix:socket' allows the user to
>>> specify a socket that will be notified whenever a session timeout
>>> occurs.  The port numbers will be written to the socket.  The socket
>>> must be created by some other application, if the socket doesn't exist
>>> then rtpproxy will continue anyway and send a warning to the log.

I think it is better to write session id (call-id, from-tag and to-tag) 
instead.

-Maxim


>>>
>>> Regards,
>>>
>>> Daniel
>>>
>>> _______________________________________________
>>> Pkg-voip-maintainers mailing list
>>> Pkg-voip-maintainers at lists.alioth.debian.org
>>> http://lists.alioth.debian.org/mailman/listinfo/pkg-voip-maintainers
>>>
>>>   
>>
>>  
>>
>