diff options
| author | Carl Baldwin <cbaldwin@digitalocean.com> | 2021-04-22 10:34:48 -0600 |
|---|---|---|
| committer | Carl Baldwin <cbaldwin@digitalocean.com> | 2021-04-22 17:40:15 -0600 |
| commit | 9e98fcf1d6bbabb1a4a815ac7d88aca044726624 (patch) | |
| tree | 2953511e8bd120aeaa3a5a621db02a3876c39352 /pkg/server/bmp.go | |
| parent | 915bfc2d81898b1e050915a20785ee2df7c025de (diff) | |
Do not suppress initial non-established peer state in MonitorPeer
The issue with the initial update was that sometimes a connection had
not yet been established. Trying to get remote and local connection
data caused a segmentation violation because `fsm.conn` was nil.
Also adds a flag to MonitorPeerRequest in the GRPC API to enable the
new behavior to avoid backward incompatibility.
fixes #2047
Diffstat (limited to 'pkg/server/bmp.go')
| -rw-r--r-- | pkg/server/bmp.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pkg/server/bmp.go b/pkg/server/bmp.go index 839dd786..67eebd0c 100644 --- a/pkg/server/bmp.go +++ b/pkg/server/bmp.go @@ -115,7 +115,7 @@ func (b *bmpClient) loop() { } if func() bool { - ops := []watchOption{watchPeerState(true)} + ops := []watchOption{watchPeerState(true, false)} if b.c.RouteMonitoringPolicy == config.BMP_ROUTE_MONITORING_POLICY_TYPE_BOTH { log.WithFields( log.Fields{"Topic": "bmp"}, |