Sip.js: Missing UA's options on SIP.WebRTC.Simple

Created on 12 Feb 2018 · 5Comments · Source: onsip/SIP.js

The server configuration guide for Asterisk sugests adding two parameters to UA's configuration:

hackIpInContact: true and
hackWssInTransport: true if using WSS.

And it works with SIP.UA but has no effect on SIP.WebRTC.Simple UA's configuration.

Can this options be added to SIP.WebRTC.Simple?

https://github.com/onsip/SIP.js/blob/3fbdab582d2cd138156768339ea4b4e0714393b1/src/WebRTC/Simple.js#L76-L89

Or if has no side effects, can all the SIP.UA configuration be made editable through the ua option in SIP.WebRTC.Simple?

var uaFixedOptions = {
  register: true,
  sessionDescriptionHandlerFactoryOptions: sessionDescriptionHandlerFactoryOptions,
};

var uaOptions = Object.assign({}, uaFixedOptions, this.options.ua);

this.ua = new SIP.UA(uaOptions);

Simple interop wontfix

Source

diegoteixeir4

Most helpful comment

I have updated our Asterisk Guide and these options are no longer needed to work with recent versions of Asterisk. I am going to close this as we do not wish to add these options to the Simple layer.

egreenmachine on 30 Apr 2018

👍2

All 5 comments

I will add some sort of Asterisk config option to Simple. The idea behind Simple is that it should be used to get started and that if you need to do advanced configuration you move to the full SIP.js API. So I am somewhat against just exposing a full blown UA configuration.

egreenmachine on 12 Feb 2018

👍2

@seanbright I'm testing with Asterisk 13.19.0. I've tried without this options and all incoming calls fail with this error:

ERROR[10695][C-000000f1]: chan_sip.c:4267 __sip_reliable_xmit: Serious Network Trouble; __sip_xmit returns error for pkt data

Unless i set hackWssInTransport: true

diegoteixeir4 on 22 Feb 2018

I use Simple with Asterisk 15.2.1 and it works without hackWssInTransport or hackIpInContact.

Hativ on 27 Apr 2018

👍1

I have updated our Asterisk Guide and these options are no longer needed to work with recent versions of Asterisk. I am going to close this as we do not wish to add these options to the Simple layer.

egreenmachine on 30 Apr 2018

👍2

Got the same issue with Asterisk 16 and 0.14 version of sipjs. No reasons and solutions over the whole internet found. So one way for me to make a fork and modify UA.js with this option manualy to continue use Simple.