[sr-dev] git:carstenbock/rtpproxy: - fixed identation

[Ovidiu Sas]

Hello Carsten,

If you want feedback, a new wiki page should give to users all the
necessary information about this new functionality.
The wiki page should provide a link to the location of the new
rtpproxy code and the tarball should be removed from the git repo.

Is there a chance to integrate this new functionality into the main
rtpproxy project?
Also, this feature should be somehow standardized as rtpproxy is used
by many SIP servers.
There may be deployments where one instance of rtpproxy can serve
simultaneously several different servers (sippy b2bua, ser, kamailio,
opensips).  How is this going to affect the functionality of this new
feature?
Another possible scenario is having two kamailio servers using the
same rtproxy: when an rtp timeout occurs, the timeout notification
will it be sent to the right sever?

Running a version of rtpproxy from git trunk with extra functionality
is not production ready.  A patch to an official release would be more
appealing for testing, but as I previously said, the best option would
be to integrate this functionality into the main rtpproxy repo.  Have
you contacted the rtpproxy team about this new functionality?


Regards,
Ovidiu Sas

On Thu, Aug 19, 2010 at 3:33 PM, Carsten Bock <carsten at bock.info> wrote:
> Hi Andrei,
> You are totally correct. It is just for temporary non-local storage.
> I wanted to do some more tests and i wanted to give others the possibility
> to test and to give me feedback.
> All changes should be fully backward compatible and the extension to the
> RTP-Proxy is automatically en-/disabled using configure and autoconf.
>
> Carsten
> 2010/8/19 Andrei Pelinescu-Onciul <andrei at iptel.org>
>>
>> On Aug 19, 2010 at 19:58, Carsten Bock <carsten at bock.info> wrote:
>> > Module: sip-router
>> > Branch: carstenbock/rtpproxy
>> > Commit: 262e55278e8b8b0e853f00465c38e3a9d13694f0
>> > URL:
>> >  http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=262e55278e8b8b0e853f00465c38e3a9d13694f0
>> >
>> > Author: Carsten Bock <lists at bock.info>
>> > Committer: Carsten Bock <lists at bock.info>
>> > Date:   Thu Aug 19 19:55:01 2010 +0200
>> >
>> > - fixed identation
>> > - added modified RTP-Proxy (rtpproxy.tar.gz, based on latest git, needs
>> > some cleanup)
>>
>> Wouldn't be better to add just a patch to the standard rtpproxy and/or
>> provide a url where one could get the modified version?
>> Having foreign tar.gz in the sr repository seems strange to me.
>>
>> > (please note, that you have to have libcurl-dev and libxmlrpc-client
>> > (same as for Kamailio-mi_xmlrpc) installed)
>> > - some simple testing files (sample config, command line parameters for
>> > pjsua (used for testing) and rtpproxy)
>>
>> Would this module work also with the standard rtpproxy?
>> Can it be compiled without the extra libs?
>> If not, forking it might be a good idea  (e.g. standard rtpproxy for
>> easy upgrade and people that don't need the extra features and modified
>>  rtpproxy with more features).
>>
>>
>> Andrei
>>
>> _______________________________________________
>> sr-dev mailing list
>> sr-dev at lists.sip-router.org
>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
>
>
>
> --
> Carsten Bock
> Schomburgstr. 80
> 22767 Hamburg
> Germany
>
> Mobile +49 179 2021244
> Home +49 40 34927217
> Fax +49 40 34927218
> mailto:carsten at bock.info
>
> _______________________________________________
> sr-dev mailing list
> sr-dev at lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
>
>