Missing UA's options on SIP.WebRTC.Simple

[diegoteixeir4]

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?

SIP.js/src/WebRTC/Simple.js

Lines 76 to 89 in 3fbdab5

	this.ua = new SIP.UA({
	// User Configurable Options
	wsServers: this.options.ua.wsServers,
	uri: this.options.ua.uri,
	authorizationUser: this.options.ua.authorizationUser,
	password: this.options.ua.password,
	displayName: this.options.ua.displayName,
	// Undocumented "Advanced" Options
	traceSip: this.options.ua.traceSip,
	userAgentString: this.options.ua.userAgentString,
	// Fixed Options
	register: true,
	sessionDescriptionHandlerFactoryOptions: sessionDescriptionHandlerFactoryOptions,
	});

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);

[egreenmachine]

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.

[diegoteixeir4]

@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

[Hativ]

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

[egreenmachine]

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.

[bigbn]

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.