Hi Marc,
just to bring the topic on the focus...
I was thinking of a compromise solution between autodetection (which has
performance penalties - multiple rtpproxy command per message) and
configurable (to be performat and flexible, but still easy to use).
Here is my idea:
force_rtp_proxy() will accept a new flag "s" - swap creation with
confirmation (default : requests create and replies confirm).
As scripting:
route {
.....
if (ACK and has_body() )
force_rtp_proxy("s");
if (INVITE) {
if (has_body())
force_rtp_proxy();
else
force_rtp_proxy();
.....
}
on_reply{
if (isflagset(x))
force_rtp_proxy("s");
else
force_rtp_proxy();
}
and that's all... If we agree on a solution, I would like to have this
fixed on CVS ASAP.
regards,
bogdan
Marc Haisenko wrote:
On Wednesday 22 February 2006 17:21, you wrote:
IMO the solution should avoid to complex openser.cfg scripts.
If a message (request or response) does not contain an SDP,
force_rtp_proxy should return without doing anything.
If a message (request or response) contains an SDP, force_rtp_proxy should:
1. check if session exits. If yes, update the session. update
means.
2. if no sessions exists, create the session
regards
klaus
Okay, I'll submit a preliminary patch next week (I'm going to have a few days
off...) and then we can discuss again :-)
Thanks Bogdan and Klaus for your feedback !
C'ya,
Marc
_______________________________________________
Devel mailing list
Devel@openser.org
http://openser.org/cgi-bin/mailman/listinfo/devel
