-
Notifications
You must be signed in to change notification settings - Fork 24
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
Unify the way _socketsMap
is populated
#1434
Labels
Comments
This should be done after the plugin refactoring is done. |
czunker
added a commit
that referenced
this issue
Sep 26, 2023
Fixes #1434 Signed-off-by: Christian Zunker <[email protected]>
czunker
added a commit
that referenced
this issue
Oct 5, 2023
Fixes #1434 Signed-off-by: Christian Zunker <[email protected]>
czunker
added a commit
that referenced
this issue
Oct 10, 2023
Fixes #1434 Signed-off-by: Christian Zunker <[email protected]>
czunker
added a commit
that referenced
this issue
Oct 10, 2023
Fixes #1434 Signed-off-by: Christian Zunker <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What is not working as you expected it?
We currently have two different ways, to fill the
_socketsMap
:cnquery/_resources/packs/core/processes.go
Line 214 in 05e6754
and
cnquery/_resources/packs/core/port.go
Line 285 in 05e6754
The implementation in port was introduced with #531.
The current assumption is, we could use the way it is done in
port.go
also inprocesses.go
to only have one way of getting the information.Where on the platform does it happen?
cnquery
The text was updated successfully, but these errors were encountered: