status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-02-22 18:38:19 +00:00
Code Issues Projects Releases Wiki Activity

Adds check to make sure port is given so we avoid a nil bind address.

Browse Source
This commit is contained in:
James Phillips 2017-10-10 18:11:21 -07:00
parent e6c50f4a71
commit 16eb2ef014
No known key found for this signature in database
GPG Key ID: 77183E682AC5FC11
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
agent/config/builder.go
View File

@ -399,6 +399,9 @@ func (b *Builder) Build() (rt RuntimeConfig, err error) {
for _, s := range c.Segments {
name := b.stringVal(s.Name)
port := b.portVal(fmt.Sprintf("segments[%s].port", name), s.Port)
if port == 0 {
return RuntimeConfig{}, fmt.Errorf("Port must be specified for segment %q", s.Name)
}
bind := b.makeTCPAddr(
b.expandFirstIP(fmt.Sprintf("segments[%s].bind", name), s.Bind),