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

Fix empty map panic in networkservice authorize.Close #1386

Merged

Conversation

NikitaSkrynnik
Copy link
Contributor

Signed-off-by: Nikita Skrynnik nikita.skrynnik@xored.com

Description

Issue link

networkservicemesh/deployments-k8s#7792

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: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
@NikitaSkrynnik NikitaSkrynnik marked this pull request as ready for review November 23, 2022 14:32
pkg/networkservice/common/authorize/server.go Outdated Show resolved Hide resolved
pkg/networkservice/common/authorize/server.go Outdated Show resolved Hide resolved
pkg/networkservice/common/authorize/server.go Outdated Show resolved Hide resolved
Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
@denis-tingaikin denis-tingaikin merged commit d83bdbd into networkservicemesh:main Nov 24, 2022
nsmbot pushed a commit to networkservicemesh/cmd-admission-webhook-k8s that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-cluster-info-k8s that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-map-ip-k8s that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-ipam-vl3 that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-init that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-vfio that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-remote-vlan that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr-proxy that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-memory that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-kernel that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-proxy-dns that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-k8s that referenced this pull request Nov 24, 2022
…k@main

PR link: networkservicemesh/sdk#1386

Commit: d83bdbd
Author: Nikita Skrynnik
Date: 2022-11-24 17:49:29 +0700
Message:
  - Fix empty map panic in networkservice authorize.Close (#1386)
* fix empty map panic in networkservice authorize.Close

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* rerun CI

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>

* apply review comments

Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
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.

2 participants