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

Do not send unnecessary message #561

Merged

Conversation

ArnoStiefvater
Copy link
Member

@ArnoStiefvater ArnoStiefvater commented Jul 14, 2021

What:

Do not send unnecessary message anymore.
When 0 alive hosts are left to scan and max_scan_hosts limit is reached no message will be generated for that case anymore.

Why:

SC-298

How:

Start a scan with 1 host and max_scan_hosts = 1 and test_alive_hosts_only = yes. Before PR a Error Message message about 0 left alive hosts is generated. After PR no message in generated anymore.

Checklist:

Fix info msg when 0 alive hosts are left to scan
and max_scan_hosts limit is reached. No message
will be generated for that case anymore.
@ArnoStiefvater ArnoStiefvater requested review from jjnicola and a team July 14, 2021 05:19
@ArnoStiefvater ArnoStiefvater added backport-to-stable This pull request will be backported to the 21.04 branch backport-to-main This pull request will be ported to the master branch labels Jul 14, 2021
@ArnoStiefvater ArnoStiefvater removed the request for review from jjnicola July 14, 2021 05:49
boreas/boreas_io.c Show resolved Hide resolved
@ArnoStiefvater ArnoStiefvater merged commit 309447b into greenbone:gvm-libs-20.08 Jul 14, 2021
ArnoStiefvater added a commit that referenced this pull request Jul 14, 2021
Do not send unnecessary message (backport #561)
ArnoStiefvater added a commit that referenced this pull request Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-main This pull request will be ported to the master branch backport-to-stable This pull request will be backported to the 21.04 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants