Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#42 pua_bla/notify.c breaks RFC3265

trunk
closed
Anca Vamanu
modules (454)
5
2008-11-04
2008-11-03
Norm Brandinger
No

pua_bla/notify.c requires the Subscription-State header value "active" to have an "expires" parameter and throws an error if it's not present.

RFC3265 (the important section listed below indicates that the "expires" is optional by use of the conditional "If".

RFC3265:

3.2.4. Subscriber NOTIFY Behavior

... If the header also contains an "expires" parameter, the subscriber ...

Note that Polycom phones appear to contain an "expires" parameter while Aastra phones to not.

Regards,
Norm

Discussion

  • Also, draft-anil-sipping-bla-04 shows some flow examples in which Subscription-State header doesn't contain "expires" parameter:

    [Page 16]

    F9 Appearance Agent ----> Alice

    NOTIFY sip:alice@ua1.example.com SIP/2.0
    From: <sip:sa@stateagent.example.com>;tag=1636248422222257
    To: <sip:alice@example.com>;tag=925A3CAD-CEBB276E
    Call-ID: ef4704d9-bb68aa0b-474c9d94@ua1.example.com
    CSeq: 2 NOTIFY
    Via: SIP/2.0/UDP stateagent.example.com;branch=z9hG4bK1846984327225734
    Max-Forwards: 70
    Content-Type: application/dialog-info+xml
    Event: dialog;ma
    Subscription-State: active
    Contact: <sip:sa@stateagent.example.com>
    Content-Length: 162

    <?xml version="1.0"?>
    <dialog-info xmlns="urn:ietf:params:xml:ns:dialog-info"
    version="40"
    state="full"
    entity="sip:alice@example.com">
    </dialog-info>

     
  • Anca Vamanu
    Anca Vamanu
    2008-11-04

    Hi Norman,

    Please test and reply with the results.

    regards,
    Anca

     
  • Anca Vamanu
    Anca Vamanu
    2008-11-04

    • assigned_to: nobody --> anca_vamanu
     
  • The fix by Anca resolves this issue.

    Thank you.
    Norm

     
    • status: open --> closed