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

⚡ speed up ports under linux #1925

Merged
merged 1 commit into from
Sep 26, 2023
Merged

⚡ speed up ports under linux #1925

merged 1 commit into from
Sep 26, 2023

Conversation

arlimus
Copy link
Member

@arlimus arlimus commented Sep 26, 2023

This is a preliminary step: One of the main resources ports is slow is because it is trying to find the process that is connected to each port. This can take a while.

This change is only an initial step, but it makes the process part async and optional. If all you need is a list of open ports, the request is now very fast on linux. It only slows down when you request the process.

We still use the process in the defaults. Additionally the process detection can be massively improved on linux...

This is a preliminary step: One of the main resources ports is slow is because it is trying to find the process that is connected to each port. This can take a while.

This change is only an initial step, but it makes the process part async and optional. If all you need is a list of open ports, the request is now very fast on linux. It only slows down when you request the process.

We still use the process in the defaults. Additionally the process detection can be massively improved on linux...

Signed-off-by: Dominik Richter <[email protected]>
@chris-rock chris-rock merged commit 11c62bc into main Sep 26, 2023
10 checks passed
@chris-rock chris-rock deleted the dom/port-process branch September 26, 2023 08:00
@github-actions github-actions bot locked and limited conversation to collaborators Sep 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants