narayana/go-xmpp
2
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Only do ensurePort for XMPP transport

Browse source 
If we always do this the address gets mangled, breaking the transport
selection.
This commit is contained in:
Wichert Akkerman 2019-10-12 15:49:22 +02:00 committed by Mickaël Rémond
parent a189748b9c
commit 8fb3e33a1f
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
client.go
View file

@ -133,8 +133,6 @@ func NewClient(config Config, r *Router) (c *Client, err error) {
} }
} }
} }
config.Address = ensurePort(config.Address, 5222)
c = new(Client) c = new(Client)
c.config = config c.config = config
c.router = r c.router = r

2
transport.go
View file

@ -39,5 +39,7 @@ func NewTransport(config TransportConfiguration) Transport {
if strings.HasPrefix(config.Address, "ws:") || strings.HasPrefix(config.Address, "wss:") { if strings.HasPrefix(config.Address, "ws:") || strings.HasPrefix(config.Address, "wss:") {
return &WebsocketTransport{Config: config} return &WebsocketTransport{Config: config}
} }
config.Address = ensurePort(config.Address, 5222)
return &XMPPTransport{Config: config} return &XMPPTransport{Config: config}
} }