Hello,

On 12/17/09 10:15 AM, Klaus Darilion wrote:


Santiago Gimeno wrote:
Hi Daniel,


El 16 de diciembre de 2009 17:30, Daniel-Constantin Mierla <mico...@gmail.com <mailto:mico...@gmail.com>> escribió:

    Hello Francisco,

    somehow I missed that patch, thanks for reminder! Is it against
    sip-router or kamailio 1.5.x?


I'm a Francisco's workmate. The patch is against kamailio 1.5. We have been using this patch in our test environment for a couple of months without problems.

BTW, we have noticed that in the dialog-info+xml body the <target uri="..."> is set to the same value as the <identity> for both local and remote elements. Is this correct? or should they be set to the local and remote contact instead?

Hi Santi!

I think you are correct. I can not remember why I implemented it different. Are the contact URIs available in the dialog structure? Then it can easily be fixed.
contacts uri are in the structure, in 'contact' array field -- they are needed to build the BYEs.

Cheers,
Daniel



regards
klaus


Best regards,

Santi



    I checked it quickly and the only thing that does not seem ok is how
    the r-uri is taken in modules/dialog/dlg_handlers.c:

    - instead of:

    +    if(parse_orig_ruri(msg)< 0) {
    +        LM_ERR("bad request or missing RURI\n");
    +                return -1;
    +    }
    +

    should be:

    +    if(parse_sip_msg_uri(msg)< 0) {
    +        LM_ERR("bad request or missing RURI\n");
    +                return -1;
    +    }
    +

    and instead of:


    + &(msg->first_line.u.request.uri) );

    should be

    +        GET_RURI(msg) );


    This ensures that latest R-URI value is taken -- you used to get
    original r-uri.

    If someone can test and report, I will commit quickly.

    Cheers,
    Daniel






    On 12/16/09 4:07 PM, Francisco Javier Lizarán Vilches wrote:
    -----Mensaje original-----
De: users-boun...@lists.kamailio.org <mailto:users-boun...@lists.kamailio.org> [mailto:users-boun...@lists.kamailio.org]
    En nombre de Daniel-Constantin Mierla
    Enviado el: miércoles, 16 de diciembre de 2009 10:38
    Para: David
    CC: users@lists.kamailio.org <mailto:users@lists.kamailio.org>
    Asunto: Re: [Kamailio-Users] Presence_Dialoginfo versioning



    On 12/15/09 4:37 PM, David wrote:
OK, it turns out that the presence application is properly updating subscriptions within a dialog, and creating new subscriptions outside
    a dialog.

The difficultly is that I am rewriting the To: header, since I used dirty tools, it was dropping ;tag=, so the server thought it was a new
    dialog and the phone the same dialog.
This should be fixed once r-uri is used instead of To header, right? I am trying to find some time to fix it, maybe with a mod param option.

    Cheers,
    Daniel
    Hi Daniel,

Some time ago we posted a patch to try to accomplish what you mention: http://sip-router.org/tracker/index.php?do=details&task_id=18 <http://sip-router.org/tracker/index.php?do=details&task_id=18>

    Hope it helps.

    Regards:
    Francisco

    I am testing to make sure that the issue is resolved.

    David



    On 2009-12-15 04:12, Olle E. Johansson wrote:
    15 dec 2009 kl. 09.59 skrev Daniel-Constantin Mierla:

    Hello,

On 12/15/09 2:20 AM, kamailio....@spam.lublink.net <mailto:kamailio....@spam.lublink.net> wrote:
    Alright, I finally found the proper RFC,
    http://www.rfc-editor.org/rfc/rfc4235.txt

    Section 4.1 :

    "version: This attribute allows the recipient of dialog
information documents to properly order them. Versions start at 0, and increment by one for each new document sent to a subscriber.
    Versions are scoped within a subscription. Versions MUST be
    representable using a non-negative 32 bit integer."

    Versions are scoped within a subscription, so when a new
subscription is started, ( after the 1 hour expiry ), the version should be reset as it is a new subscription and therefore a new
    scope ?

    When the subscription expires, is it renewed or is a new
    subscription created? Is the scope separate, or is it the same
    subscription updated?
I think this is another questionable thing about SIP. IMO, it is same subscription if the dialog attributes do not change (call-id, from tag and to tag). But others can argue is it a new subscription.
    Anyone else on this one?

The proper RFC for generic subscription/notify questions is RFC 3265.

    "3.1.1 Subscription Duration
SUBSCRIBE requests SHOULD contain an Expires header (defined in SIP
    [2]). This expires value indicates
    the duration of the subscription. In order to keep subscriptions
    effective beyond the duration communicated
in the Expires header, subscribers need to refresh subscriptions on a
    periodic basis using a new
    SUBSCRIBE message on the same dialog as defined in SIP [2]"

This indicates to me that it's the same subscription as long as you
    refresh it.

RFC4235 refers to RFC 3265 for general terminology about subscriptions.

    /O
    _______________________________________________
    Kamailio (OpenSER) - Users mailing list
    Users@lists.kamailio.org <mailto:Users@lists.kamailio.org>
    http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
    http://lists.openser-project.org/cgi-bin/mailman/listinfo/users

    --
    Daniel-Constantin Mierla
    * http://www.asipto.com/


    _______________________________________________
    Kamailio (OpenSER) - Users mailing list
    Users@lists.kamailio.org <mailto:Users@lists.kamailio.org>
    http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
    http://lists.openser-project.org/cgi-bin/mailman/listinfo/users
    _______________________________________________
    Kamailio (OpenSER) - Users mailing list
    Users@lists.kamailio.org <mailto:Users@lists.kamailio.org>
    http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
    http://lists.openser-project.org/cgi-bin/mailman/listinfo/users


    --     Daniel-Constantin Mierla
    * http://www.asipto.com/


    _______________________________________________
    Kamailio (OpenSER) - Users mailing list
    Users@lists.kamailio.org <mailto:Users@lists.kamailio.org>
    http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
    http://lists.openser-project.org/cgi-bin/mailman/listinfo/users



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

_______________________________________________
Kamailio (OpenSER) - Users mailing list
Users@lists.kamailio.org
http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
http://lists.openser-project.org/cgi-bin/mailman/listinfo/users

_______________________________________________
Kamailio (OpenSER) - Users mailing list
Users@lists.kamailio.org
http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
http://lists.openser-project.org/cgi-bin/mailman/listinfo/users


--
Daniel-Constantin Mierla
* http://www.asipto.com/


_______________________________________________
Kamailio (OpenSER) - Users mailing list
Users@lists.kamailio.org
http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
http://lists.openser-project.org/cgi-bin/mailman/listinfo/users

Reply via email to