Summary:	ASTERISK-15355: [patch] port in users.conf is not honored in the register statement
Reporter:	Johann Steinwendtner (steinwej)	Labels:
Date Opened:	2009-12-22 14:14:32.000-0600	Date Closed:	2011-06-07 14:08:17
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Channels/chan_sip/Registration
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) users_port.diff
Description:	VoIP provider has SIP port set to 9060 The account for the provider is defined in users.conf as follows: [testuser] context = DID_test host = voipprovider.com port = 9060 username = testuser fromuser = testuser fromdomain = secret = verysecret insecure = no trunkname = tname hasiax = no registeriax = no hassip = yes registersip = yes trunkstyle = voip hasexten = no disallow = all allow = alaw,gsm,adpcm it is expected that the account testuser registers at voipprovider.com. But registration fails because * send the REGISTER to the default SIP port 5060
Comments:	By: David Woolley (davidw) 2009-12-23 06:12:09.000-0600 Does this change also affect sip.conf, as there is documentation in sip.conf.sample about port not working when specified in the device section, rather than a register line, which may need amending. By: Johann Steinwendtner (steinwej) 2009-12-23 07:06:25.000-0600 No, this does only apply to devices loaded from users.conf. By: David Woolley (davidw) 2009-12-23 07:54:07.000-0600 I assume you meant sip.conf, just then. By: Johann Steinwendtner (steinwej) 2010-01-01 13:14:07.000-0600 ahm, no I meant users.conf. Maybe I have misunderstood your question. The patch appends the port to the host where the register is sent to. By: Leif Madsen (lmadsen) 2010-01-04 10:11:45.000-0600 Does the users.conf.sample file show an example of the usage of 'port'? By: Johann Steinwendtner (steinwej) 2010-01-04 10:19:04.000-0600 No, as per trunk, there is no example of the usage of 'port'. By: Leif Madsen (lmadsen) 2010-01-04 14:23:24.000-0600 Is this adding a new feature then, or is this fixing an existing, undocumented feature? Regardless, the patch should also include the changes to the users.conf.sample file as well. By: Johann Steinwendtner (steinwej) 2010-01-08 10:03:46.000-0600 Well, then it would be a new feature. I will try. By: Leif Madsen (lmadsen) 2010-03-23 09:51:46 Suspending this issue due to a lack of response from the reporter. If the reporter is interested in moving this issue forward, then please reopen the issue and provide the requested information. Thanks!