status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-01-10 22:06:20 +00:00
Code Issues Projects Releases Wiki Activity

Fix listener error handling

Browse Source
This commit is contained in:
freddygv 2020-04-16 19:08:41 -06:00
parent 6abc71f915
commit d82e7e8c2a
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
agent/xds/listeners.go
View File

@ -259,7 +259,9 @@ func (s *Server) listenersFromSnapshotGateway(cfgSnap *proxycfg.ConfigSnapshot,
return nil, err return nil, err
} }
} }
resources = append(resources, l) if l != nil {
resources = append(resources, l)
}
} }
return resources, err return resources, err
} }
@ -600,6 +602,9 @@ func (s *Server) makeTerminatingGatewayListener(name, addr string, port int, cfg
} }
err = injectConnectFilters(cfgSnap, token, l, false) err = injectConnectFilters(cfgSnap, token, l, false)
if err != nil {
return nil, fmt.Errorf("failed to inject authz filer: %v", err)
}
// This fallback catch-all filter ensures a listener will be present for health checks to pass // This fallback catch-all filter ensures a listener will be present for health checks to pass
// Envoy will reset these connections since known endpoints are caught by filter chain matches above // Envoy will reset these connections since known endpoints are caught by filter chain matches above