Skip to content

Commit

Permalink
Fix: Host already had the port attached
Browse files Browse the repository at this point in the history
This happened when providing a port number in the URI as the `Host`
field still has the port number while the `Hostname()` method only
returns the hostname without the port.

fixes #33

Signed-off-by: Knut Ahlers <[email protected]>
  • Loading branch information
Luzifer committed Jan 17, 2019
1 parent 085a158 commit 3cd5ab3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion auth_ldap.go
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ func (a authLDAP) dial() (*ldap.Conn, error) {
return nil, err
}

host := u.Host
host := u.Hostname()
port := u.Port()

var l *ldap.Conn
Expand Down

0 comments on commit 3cd5ab3

Please sign in to comment.