[OpenSIPS-Devel] [ opensips-Bugs-2641464 ] engage_media_proxy failes if called after create_dialog

SourceForge.net noreply at sourceforge.net
Fri Feb 27 16:46:15 CET 2009


Bugs item #2641464, was opened at 2009-02-26 14:16
Message generated for change (Comment added) made by bogdan_iancu
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086410&aid=2641464&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Thomas Gelf (thomas_gelf)
Assigned to: Nobody/Anonymous (nobody)
Summary: engage_media_proxy failes if called after create_dialog

Initial Comment:
If I correctly understood mediaproxy.c it installs a callback handler __dialog_created, responsible for setting up the other handlers for this dialog (using dlg_api.register_dlgcb).

However, if you called create_dialog() before, "dialog_created" will not be fired again if you are using engage_media_proxy().

Proposed solution: either add a note to current documentation ("...do NOT use create_dialog when using engage_media_proxy...") or find a way to catch this situation in EngageMediaProxy (maybe: if dlg started, immediately register dialog callbacks).

Best regards,
Thomas Gelf


----------------------------------------------------------------------

>Comment By: Bogdan-Andrei Iancu (bogdan_iancu)
Date: 2009-02-27 17:46

Message:
But when calling the create_dialog() function, the dialog module is
automatically triggering the dialog_created callback.
Internally there is the same function tiggered by flag setting or by the
create function.

Maybe the engage_media_proxy() is expecting to have the dialog created
later and it is doing some other stuff before.

Regards,
Bogdan

----------------------------------------------------------------------

Comment By: Dan (dan_pascu)
Date: 2009-02-27 12:48

Message:
I think that the dialog module should immediately trigger the registered
callback for create dialog, if the dialog was created before that point.
This is the most obvious thing to do, as the code using the dialog module
will not have to be aware of the internal details of the dialog module and
its state, nor will it need to inspect the dialog module to figure out what
to do next.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086410&aid=2641464&group_id=232389



More information about the Devel mailing list