Skip to content

Commit

Permalink
Backport of Fix error handling in Client consulDiscoveryImpl into rel…
Browse files Browse the repository at this point in the history
…ease/1.2.x (#14454)

This pull request was automerged via backport-assistant
  • Loading branch information
hc-github-team-nomad-core authored Sep 2, 2022
1 parent ef1d305 commit 7b739ae
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .changelog/14431.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:bug
client: Fixed bug where clients could attempt to connect to servers with invalid addresses retrieved from Consul.
```
1 change: 1 addition & 0 deletions client/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -2793,6 +2793,7 @@ DISCOLOOP:
addr, err := net.ResolveTCPAddr("tcp", p)
if err != nil {
mErr.Errors = append(mErr.Errors, err)
continue
}
srv := &servers.Server{Addr: addr}
nomadServers = append(nomadServers, srv)
Expand Down

0 comments on commit 7b739ae

Please sign in to comment.