[OpenSER-Devel] [ openser-Patches-1887826 ] [MSILO] Some improvements

[SourceForge.net]

Patches item #1887826, was opened at 2008-02-06 12:01
Message generated for change (Comment added) made by amr42
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1887826&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: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Iñaki Baz (ibc_sf)
Assigned to: Nobody/Anonymous (nobody)
Summary: [MSILO] Some improvements

Initial Comment:
I've been playing a bit with msilo.c code and changed some things to make it more flexible. The changes are made from SVN rev 3632.


This is my changelog:

- Notification "From" is now the original destination ("registrar" parameter deleted). I think this is better since the sender won't see a new IM window 
from a strange user when OpenSer sends back the notification. Instead he will see it in the same IM window.

- Added "offline_message" parameter allowing HTML. So the admin could set him own notification message in openser.cfg. If not set, the default value is:
  "<em>I'm offline. The message will be delivered when I'm online.</em>"

- Deleted "Contact" header in notification (not necessary in MESSAGE since MESSAGE doesn't establish a dialog).

- Deleted CONTACT* and OFFLINE_MESSAGE* #defines and buf1[1024] (not necessary now).

- Added HEADERS #define to set "Content-Type" header as "text/html". I've not made "Content-Type" as a parameter since "text/html" accepts plain text perfectly, but it could be a new "notification_content_type" parameter.

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

Comment By: Aron Rosenberg (amr42)
Date: 2008-02-06 17:19

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

Several comments:

1. Removing the contact address and the registrar address removes the
ability for an endpoint client to tell that the message was queued. (Aside
from Date header). If the contact address is your registrar (which is why
its called that) then your client can know that it was a queued offline
message. Counterpath does the wrong thing (they use the Contact address as
the "From" instead of the From Uri)

2. Adding the "offline_message" param is not the ideal way to go about
solving this problem. When a message is queued for delivery a 202 is
supposed to be returned to the sending client. 202 indicates that this
message was accepted but unable to be end to end delivered. Proper clients
should show the "This user is offline" as a result of the 202.

3. You mention that the Notification "From" is now the original
destination, do you mean that the original From Uri is now the "To" on the
message being delivered from the SILO? This seems completely wrong. See #1,
but your workaround while fixing counterpath breaks other compliant SIP
endpoints which referenace the To Uri and From Uri (rather than Contact or
Via or others)

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

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