mainThe eXtended osip library - Support: sr #108455, SUBSCRIBE request does not contain...

 
 

sr #108455: SUBSCRIBE request does not contain Accept header

Submitter:  None
Submitted:  Thu 12 Dec 2013 07:15:17 PM UTC
   
 
Category:  None Priority:  5 - Normal
Severity:  3 - Normal Status:  Invalid
Privacy:  Public Assigned to:  aymeric
Originator Email:  -email is unavailable- Open/Closed:  Closed
Operating System:  GNU/Linux
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 13 Dec 2013 12:30:16 AM UTC, comment #1: 

Hi,

Tks for your report.

There exists 2 ways to add any SIP headers in SIP message being built with eXosip2.

The To header of eXosip_subscribe_build_initial_request and similar API can contains URL PARAMETERS (request->to->url->url_headers)

Such header will be added automatically into SIP message. For example: sip:xxxx@domain.org?Subject=Hello

The other way from the external app is to first
1/ build the SIP message.
   eXosip_subscribe_build_initial_request (&subscribe, url, identity, proxy_route, event, expires);
2/ add your header in the returned sip message:
   osip_message_set_header (*sub, "Accept", "application/pidf+xml");
3/ send SIP message:
   eXosip_subscribe_send_initial_request (subscribe);

eXosip2 must remains flexible and cannot integrate header that might not be required for some users/developers.

Tks
Aymeric

Aymeric MOIZARD <aymeric>
Group administrator
Thu 12 Dec 2013 07:15:17 PM UTC, original submission:  

Using Linphone 3.6.1 with exosip2 4.0.0. SUBSCRIBE requests do not contain an Accept header, which causes all contacts to be shown as offline. Asterisk spits out the following warning:
WARNING[55098]: chan_sip.c:25470 handle_request_subscribe: SUBSCRIBE failure: no Accept header: pvt: stateid: -1, laststate: 0, dialogver: 0, subscribecont: '', subscribeuri: ''

Was able to get it working by simply adding a line in eXsubscription_api.c to have it send a valid header. Patch is attached.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #29889:  exosip-accept.patch added by None (450B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by aymeric (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-12-13 aymeric StatusNone Invalid
        Assigned toNone aymeric
        Open/ClosedOpen Closed
    2013-12-12 None Attached File- Added exosip-accept.patch, #29889

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code