[Kamailio-Devel] [ openser-Patches-1824748 ] Add callID testing to registrar:registered()

SourceForge.net noreply at sourceforge.net
Mon Sep 1 13:25:52 CEST 2008


Patches item #1824748, was opened at 2007-11-02 16:32
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1824748&group_id=139143

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: ver devel
Status: Closed
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Kobi Eshun (ekobi)
Assigned to: Henning Westerholt (henningw)
Summary: Add callID testing to registrar:registered()

Initial Comment:
Attached is a small patch to facilitate managing multiple registration instances using the same credentials. It introduces a new module AVP, reg_callid_avp. If reg_callid_avp is defined and populated when registered() is invoked, the result is TRUE only if an active registration with the specified callID is found.

--
kobi

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

Comment By: Nobody/Anonymous (nobody)
Date: 2008-09-01 11:25

Message:
Logged In: NO 

Hi Kobi!

I wonder what is the use case for this feature?

klaus

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

Comment By: Henning Westerholt (henningw)
Date: 2008-08-29 16:12

Message:
Logged In: YES 
user_id=337916
Originator: NO

I fixed some small error in the pv parsing and added the description
you've given. I did some tests, it seems to work correctly. Would be good
if you could do also some tests, i commited it to trunk.

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

Comment By: Henning Westerholt (henningw)
Date: 2008-08-29 15:00

Message:
Logged In: YES 
user_id=337916
Originator: NO

Thanks for the updates, i'll merge this patch with some small adaptions.

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

Comment By: Kobi Eshun (ekobi)
Date: 2008-08-29 00:14

Message:
Logged In: YES 
user_id=1039134
Originator: YES

Revised again to resolve bad refs to LOG macros and xl_XXX() funcitons.
File Added: registrar_callid_test.patch

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

Comment By: Kobi Eshun (ekobi)
Date: 2008-08-28 17:49

Message:
Logged In: YES 
user_id=1039134
Originator: YES

My bad -- grabbed the wrong version. Try this one. Cheers,
--
kobi
File Added: registrar_callid_test.patch

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

Comment By: Henning Westerholt (henningw)
Date: 2008-08-28 13:24

Message:
Logged In: YES 
user_id=337916
Originator: NO

Hi Kobi, i've tried, but get still this errors. It seems that both patches
differs only in one comment and line break.

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

Comment By: Kobi Eshun (ekobi)
Date: 2008-08-27 20:49

Message:
Logged In: YES 
user_id=1039134
Originator: YES

Adding updated version that compiles against 1.4. Be aware that it has not
been tested in this configuration.
File Added: registrar_callid_test.patch

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

Comment By: Henning Westerholt (henningw)
Date: 2008-08-11 10:21

Message:
Logged In: YES 
user_id=337916
Originator: NO

Hi,

thank you for the patch. I've tried it, it did not compile anymore for the
actual trunk. I get the following errors:

reg_mod.c: In function mod_init:
reg_mod.c:278: warning: implicit declaration of function
xl_parse_spec
reg_mod.c:279: Error: error: XL_THROW_ERROR undeclared (first use in
this function)
reg_mod.c:279: Error: error: (Each undeclared identifier is reported only
once
reg_mod.c:279: Error: error: for each function it appears in.)
reg_mod.c:279: Error: error: XL_DISABLE_MULTI undeclared (first use
in this function)
reg_mod.c:279: Error: error: XL_DISABLE_COLORS undeclared (first use
in this function)
reg_mod.c:280: Error: error: XL_AVP undeclared (first use in this
function)
reg_mod.c:281: warning: implicit declaration of function LOG
reg_mod.c:286: warning: implicit declaration of function
xl_get_avp_name

Perhaps do you have already created an updated version that you use
internally? Cheers,

Henning

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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1824748&group_id=139143



More information about the Devel mailing list