Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[qfix] Check if error is nil in proxy listener #407

Merged

Conversation

Bolodya1997
Copy link

Description

Continues #403 and fixes introduced error.

Motivation

If err is nil ok will also be false.

How Has This Been Tested?

  • Added unit testing to cover
  • Tested manually
  • Tested by integration testing
  • Have not tested

Types of changes

  • Bug fix
  • New functionallity
  • Documentation
  • Refactoring
  • CI

Signed-off-by: Vladimir Popov <vladimir.popov@xored.com>
@denis-tingaikin denis-tingaikin merged commit a57956d into networkservicemesh:main Oct 13, 2021
nsmbot pushed a commit to networkservicemesh/cmd-forwarder-vpp that referenced this pull request Oct 13, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#407

Commit: a57956d
Author: Vladimir Popov
Date: 2021-10-13 13:06:05 +0700
Message:
  - Check if error is nil in proxy listener (#407)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-icmp-responder-vpp that referenced this pull request Oct 13, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#407

Commit: a57956d
Author: Vladimir Popov
Date: 2021-10-13 13:06:05 +0700
Message:
  - Check if error is nil in proxy listener (#407)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-vpp that referenced this pull request Oct 13, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#407

Commit: a57956d
Author: Vladimir Popov
Date: 2021-10-13 13:06:05 +0700
Message:
  - Check if error is nil in proxy listener (#407)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-firewall-vpp that referenced this pull request Oct 13, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#407

Commit: a57956d
Author: Vladimir Popov
Date: 2021-10-13 13:06:05 +0700
Message:
  - Check if error is nil in proxy listener (#407)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
Mixaster995 added a commit to Mixaster995/sdk-vpp that referenced this pull request Oct 13, 2021
Mixaster995 added a commit to Mixaster995/sdk-vpp that referenced this pull request Oct 13, 2021
…)"

This reverts commit a57956d.

Signed-off-by: Mikhail Avramenko <avramenkomihail15@gmail.com>
denis-tingaikin pushed a commit that referenced this pull request Oct 13, 2021
* Revert "Check if error is nil in proxy listener (#407)"

This reverts commit a57956d.

Signed-off-by: Mikhail Avramenko <avramenkomihail15@gmail.com>

* Revert "Fix error handling in memifproxy (#403)"

This reverts commit 792a420.

Signed-off-by: Mikhail Avramenko <avramenkomihail15@gmail.com>
nsmbot pushed a commit that referenced this pull request Jan 24, 2022
…k-kernel@main

PR link: networkservicemesh/sdk-kernel#407

Commit: 2292537
Author: Denis Tingaikin
Date: 2022-01-24 19:19:23 +0300
Message:
  - Merge pull request #407 from glazychev-art/iprule
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants