fix: use default port definitions

InnovativeUS · Nov 27, 2024 · 7304a2c · 7304a2c
1 parent e11ca47
commit 7304a2c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/sip/parse_uri_test.go b/sip/parse_uri_test.go
@@ -50,6 +50,7 @@ func TestParseUri(t *testing.T) {
 			assert.Equal(t, "alice", uri.User)
 			assert.Equal(t, "atlanta.com", uri.Host)
 			assert.False(t, uri.IsEncrypted())
+			assert.Equal(t, "atlanta.com:5060", uri.HostPort())
 		}
 
 		testCases = []string{
@@ -63,6 +64,7 @@ func TestParseUri(t *testing.T) {
 			assert.Equal(t, "alice", uri.User)
 			assert.Equal(t, "atlanta.com", uri.Host)
 			assert.True(t, uri.IsEncrypted())
+			assert.Equal(t, "atlanta.com:5061", uri.HostPort())
 		}
 
 	})

diff --git a/sip/uri.go b/sip/uri.go
@@ -145,6 +145,16 @@ func (uri *Uri) Addr() string {
 
 // HostPort represents host:port part
 func (uri *Uri) HostPort() string {
-	p := strconv.Itoa(uri.Port)
+	transport := TransportTCP
+	if uri.IsEncrypted() {
+		transport = TransportTLS
+	}
+
+	port := uri.Port
+	if port == 0 {
+		port = DefaultPort(transport)
+	}
+
+	p := strconv.Itoa(port)
 	return uri.Host + ":" + p
 }