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

Suggest adding multi port detection #3265

Closed
1 task done
yangyzp opened this issue Jun 14, 2023 · 2 comments
Closed
1 task done

Suggest adding multi port detection #3265

yangyzp opened this issue Jun 14, 2023 · 2 comments
Labels
area:monitor Everything related to monitors feature-request Request for new features to be added

Comments

@yangyzp
Copy link

yangyzp commented Jun 14, 2023

⚠️ Please verify that this feature request has NOT been suggested before.

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

New Monitor

🔖 Feature description

Adding multi port detection to a detection task,Simultaneously detecting multiple ports.
As shown in the following figure:
https://s2.loli.net/2023/06/14/b76YVq8nx3PjcMp.png
Snipaste_2023-06-14_13-06-18.png

✔️ Solution

Adding multi port detection to a detection task,Simultaneously detecting multiple ports.

@yangyzp yangyzp added the feature-request Request for new features to be added label Jun 14, 2023
@CommanderStorm
Copy link
Collaborator

But one monitor should only monitor one thing => monitoring more than one thing with one monitor violates the debuggabiltiy of the system.
I think using multiple monitors in a monitor group (new since 1.22.0-beta.0) would be a better option.

@CommanderStorm CommanderStorm added the area:monitor Everything related to monitors label Dec 5, 2023
@CommanderStorm
Copy link
Collaborator

@yangyzp
We are consolidating duplicate issues a bit to make issue management easier.
I think, we should track this issue in #497 as there is no functional difference (maybe just small naming differences, but nothing that would require a different issue imo)
⇒ I am going to close this as a duplicate.

@CommanderStorm CommanderStorm closed this as not planned Won't fix, can't repro, duplicate, stale Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:monitor Everything related to monitors feature-request Request for new features to be added
Projects
None yet
Development

No branches or pull requests

2 participants