[OpenSER-Devel] [ openser-Bugs-1768774 ] pua_set_publish()+save() clears AVPs

[SourceForge.net]

Bugs item #1768774, was opened at 2007-08-06 22:21
Message generated for change (Comment added) made by miconda
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1768774&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 1.2.x
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Kobi Eshun (ekobi)
Assigned to: Daniel-Constantin Mierla (miconda)
Summary: pua_set_publish()+save() clears AVPs

Initial Comment:
Hi,

The pua_set_publish() functionality provided by the pua_usrloc module appears to clear AVPs. Consider the following script snippet:

    $avp(i:900)="I am an AVP, and I exist...";
    xlog("L_CRIT", "avp(i:900) before save(location): $avp(i:900)");
    pua_set_publish();
    save("location");
    xlog("L_CRIT", "avp(i:900) after save(location): $avp(i:900)");

and its corresponding log output:

 0(27183) avp(i:900) before save(location): I am an AVP, and I exist...
 0(27183) avp(i:900) after save(location): <null>

This behavior is observed with code from the tip of the stable branch. Cheers,
--
kobi

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

>Comment By: Daniel-Constantin Mierla (miconda)
Date: 2007-11-09 18:12

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

Fixed some time ago, in both trunk and 1.2.x

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

Comment By: Kobi Eshun (ekobi)
Date: 2007-08-21 23:48

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

Hi Daniel,

Sorry about the dropout there -- just got back from vacation.

So I applied 2592 to the 1.2 branch and it appears to work okay. Cheers,
--
kobi

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

Comment By: Aron Rosenberg (amr42)
Date: 2007-08-14 19:33

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

We will test next week to see if the fix worked which should hopefully be
in time for the 1.2.2 release.

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

Comment By: Daniel-Constantin Mierla (miconda)
Date: 2007-08-10 17:03

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

I have applied a patch to devel version -- the issue was in tm module. Can
you test and see if works ok, then I will backport to 1.2. Thanks, Daniel.

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

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