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

Add method to check if a target finished cleanly or crashed. #133

Merged
merged 1 commit into from
Sep 18, 2019

Conversation

jjnicola
Copy link
Member

Some scans can finish cleanly even if the don't scan all the hosts in the target string.
This produced to have an ended scan with a incomplete progress scan percentage (< 100%), which was treated as an error. Now this function allow to do a check and to decide if it is an error or the scan finished cleanly.

Some scans can finish cleanly even if the don't scan all the hosts in the target string.
This produced to have an ended scan with a incomplete progress scan percentage (< 100%), which was treated
as an error. Now this function allow to do a check and to decide if it is an error or the scan finished cleanly.
@codecov
Copy link

codecov bot commented Sep 17, 2019

Codecov Report

Merging #133 into master will decrease coverage by 0.14%.
The diff coverage is 14.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #133      +/-   ##
==========================================
- Coverage   68.85%   68.71%   -0.15%     
==========================================
  Files          12       12              
  Lines        1737     1742       +5     
==========================================
+ Hits         1196     1197       +1     
- Misses        541      545       +4
Impacted Files Coverage Δ
ospd/ospd.py 73.27% <14.28%> (-0.35%) ⬇️

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 ef1da61...bc87dd5. Read the comment docs.

@bjoernricks bjoernricks merged commit 81abad2 into greenbone:master Sep 18, 2019
@jjnicola jjnicola deleted the finished-target branch September 25, 2019 10:12
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