Skip to content
This repository has been archived by the owner on Nov 29, 2021. It is now read-only.

Allow the scanner to update total count of hosts. #332

Merged
merged 3 commits into from
Oct 12, 2020

Conversation

jjnicola
Copy link
Member

@jjnicola jjnicola commented Oct 8, 2020

What:
Allow the scanner to update total count of hosts.

Why:
In some cases it can differ to the one calculated from original target string
E.g. OpenVAS will remove duplicated hosts, unresolved hostname and
add new hosts in cases a hostname resolves to more than one IP address

How:
Run a scan against a hostname target which resolves to more than one ip address.

Checklist:

In some cases it can differ to the one calculated from original target string
E.g. OpenVAS will remove duplicated hosts, unresolved hostname and
add new hosts in cases a hostname resolves to more than one IP address
@codecov
Copy link

codecov bot commented Oct 8, 2020

Codecov Report

Merging #332 into ospd-20.08 will increase coverage by 0.11%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff               @@
##           ospd-20.08     #332      +/-   ##
==============================================
+ Coverage       73.35%   73.47%   +0.11%     
==============================================
  Files              23       23              
  Lines            2560     2571      +11     
==============================================
+ Hits             1878     1889      +11     
  Misses            682      682              
Impacted Files Coverage Δ
ospd/ospd.py 71.23% <100.00%> (+0.08%) ⬆️
ospd/scan.py 92.34% <100.00%> (+0.30%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f637f0...a84d0e2. Read the comment docs.

@jjnicola jjnicola marked this pull request as ready for review October 9, 2020 09:08
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.

None yet

2 participants