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(a)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(a)iptel.org>
On Aug 19, 2010 at 19:58, Carsten Bock <carsten(a)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=262e552…
Author: Carsten Bock <lists(a)bock.info>
Committer: Carsten Bock <lists(a)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(a)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@bock.info
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev