diff options
| author |   mhalaj1 <matus.halaj@pantheon.tech> | 2021-08-26 20:15:08 +0200 |
|---|---|---|
| committer | mhalaj1 <matus.halaj@pantheon.tech> | 2021-09-07 15:24:53 +0200 |
| commit | c09ee3241377aae2530a73d48c4e20641d76d0ee (patch) | |
| tree | 219fd1f7d9ba595f0f4c8dac9796bc76ce3556fc /binapi/nsh/nsh_rpc.ba.go | |
| parent | debc52dea8a81417bb08ca5bb934c7876b6d65e0 (diff) | |
Refactoring and fixes
* refactor creation of new channel
* add missing closing of created streams
* correct documentation regarding thread safety of stream
Signed-off-by: mhalaj1 <matus.halaj@pantheon.tech>
Change-Id: Ic601efff298fcbdecaafab83fa236253af69de21
Diffstat (limited to 'binapi/nsh/nsh_rpc.ba.go')
| -rw-r--r-- | binapi/nsh/nsh_rpc.ba.go | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/binapi/nsh/nsh_rpc.ba.go b/binapi/nsh/nsh_rpc.ba.go index db79b4f..d7dd6aa 100644 --- a/binapi/nsh/nsh_rpc.ba.go +++ b/binapi/nsh/nsh_rpc.ba.go @@ -78,6 +78,10 @@ func (c *serviceClient_NshEntryDumpClient) Recv() (*NshEntryDetails, error) { case *NshEntryDetails: return m, nil case *vpe.ControlPingReply: + err = c.Stream.Close() + if err != nil { + return nil, err + } return nil, io.EOF default: return nil, fmt.Errorf("unexpected message: %T %v", m, m) @@ -117,6 +121,10 @@ func (c *serviceClient_NshMapDumpClient) Recv() (*NshMapDetails, error) { case *NshMapDetails: return m, nil case *vpe.ControlPingReply: + err = c.Stream.Close() + if err != nil { + return nil, err + } return nil, io.EOF default: return nil, fmt.Errorf("unexpected message: %T %v", m, m) |