[OpenSER-Devel] [ openser-Bugs-1863627 ] pua_usrloc: presentity table(s) not updated upon unregister

[SourceForge.net]

Bugs item #1863627, was opened at 2008-01-03 22:54
Message generated for change (Comment added) made by anca_vamanu
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1863627&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: Invalid
Priority: 5
Private: No
Submitted By: Philippe Favier (quark38)
Assigned to: anca (anca_vamanu)
Summary: pua_usrloc: presentity table(s) not updated upon unregister

Initial Comment:

Problem:
--------
PUBLISH("closed") not handled.

Analysis:
---------
The function ul_publish() sends only PUBLISH("open") messages, PUBLISH("closed") messages are never sent.

Proposed solution:
------------------
1) Add an extra parameter "publish_status" to function build_pidf(), which can be "open" or closed".
2) In ul_publish(), call build_pidf("open") instead of build_pidf() as currently.
3) In ul_publish(), add a call to build_pidf("closed") for unregistration.


Philippe Favier
http://www.ives.fr/



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

>Comment By: anca (anca_vamanu)
Date: 2008-01-06 11:58

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

Hi Philippe,

Indeed the pua_usrloc does not send any Publish messages with status
'closed' in the pidf body when unRegistering. 
Instead, when an unREGISTER is received, it sends a Publish message with
expires '0', that tells the presence server that the user has gone
off-line. 
This is what most sip clients also do. 
The Notifies sent by the presence server, however, are the same in the two
cases, having a pidf with status 'closed'. 
In conclusion, there is no reason to send a body, of course with status
'closed' when unRegistering. 
As an observation, I see the use of a body with status closed in Publish
as a means to hide the presence of a user, like 'invisible' status in
certain clients. There is no need for that here. 
As a consequence, I will close this thread as invalid. However, if you
have some other points to add that would support the request you can open
it again.

regards, 
Anca Vamanu

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

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