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

Do not send all params as scanner prefs #305

Merged
merged 3 commits into from
Jul 30, 2020

Conversation

ArnoStiefvater
Copy link
Member

Do not add all params which are in the OSPD_PARAMS dict to the params which are set as scan preferences.

Depends on:
greenbone/ospd-openvas#297

@codecov
Copy link

codecov bot commented Jul 24, 2020

Codecov Report

Merging #305 into ospd-20.08 will decrease coverage by 1.33%.
The diff coverage is 36.36%.

Impacted file tree graph

@@              Coverage Diff               @@
##           ospd-20.08     #305      +/-   ##
==============================================
- Coverage       74.69%   73.35%   -1.34%     
==============================================
  Files              23       23              
  Lines            2545     2560      +15     
==============================================
- Hits             1901     1878      -23     
- Misses            644      682      +38     
Impacted Files Coverage Δ
ospd/main.py 0.00% <0.00%> (ø)
ospd/ospd.py 71.14% <0.00%> (-4.64%) ⬇️
ospd/command/command.py 87.94% <100.00%> (+0.14%) ⬆️
ospd/network.py 72.22% <100.00%> (+0.10%) ⬆️
ospd/scan.py 92.03% <100.00%> (+0.47%) ⬆️
ospd/protocol.py 88.04% <0.00%> (-6.53%) ⬇️

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 da303ef...93a80a2. Read the comment docs.

jjnicola
jjnicola previously approved these changes Jul 29, 2020
Do not add all params which are in the OSPD_PARAMS
dict to the params which are set as preferences.
Documentation about the options we got from the
<scanner_params> XML element.
@ArnoStiefvater ArnoStiefvater merged commit bfe22fc into greenbone:ospd-20.08 Jul 30, 2020
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