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

Improve nvt feed check. #195

Merged
merged 4 commits into from
Feb 17, 2020
Merged

Improve nvt feed check. #195

merged 4 commits into from
Feb 17, 2020

Conversation

jjnicola
Copy link
Member

Compare the amount of filename keys and nvt keys in redis with the amount of oid loaded in memory.

Compare the amount of filename keys and nvt keys in redis with the amount of oid loaded in memory.
@codecov
Copy link

codecov bot commented Feb 14, 2020

Codecov Report

Merging #195 into master will decrease coverage by 0.02%.
The diff coverage is 64.7%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #195      +/-   ##
==========================================
- Coverage   75.08%   75.06%   -0.03%     
==========================================
  Files           5        5              
  Lines        1160     1175      +15     
==========================================
+ Hits          871      882      +11     
- Misses        289      293       +4
Impacted Files Coverage Δ
ospd_openvas/db.py 100% <100%> (ø) ⬆️
ospd_openvas/daemon.py 62.4% <45.45%> (-0.09%) ⬇️

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 8efe6b1...0026cf3. Read the comment docs.

- Set reverse lookup options. [#185](https://github.com/greenbone/ospd-openvas/pull/185)

- Set reverse lookup options. [#185](https://gitub.com/greenbone/ospd-openvas/pull/185)
- Check if the amount of vts in redis is coherent. [#195](https://github.com/greenbone/ospd-openvas/pull/195)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Check if the amount of vts in redis is coherent. [#195](https://github.com/greenbone/ospd-openvas/pull/195)
- Check if the amount of vts in redis is coherent. [#195](https://github.com/greenbone/ospd-openvas/pull/195)

CHANGELOG.md Outdated
@@ -11,8 +11,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
- Add typing to daemon.py, nvticache.py and db.py. [#161](https://github.com/greenbone/ospd-openvas/pull/161)[#162](https://github.com/greenbone/ospd-openvas/pull/162)[#163](https://github.com/greenbone/ospd-openvas/pull/163)
- Add support for alive test settings. [#182](https://github.com/greenbone/ospd-openvas/pull/182)
- Add missing scan preferences expand_vhosts and test_empty_vhost. [#184](https://github.com/greenbone/ospd-openvas/pull/184)
- Set reverse lookup options. [#185](https://github.com/greenbone/ospd-openvas/pull/185)

- Set reverse lookup options. [#185](https://gitub.com/greenbone/ospd-openvas/pull/185)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Set reverse lookup options. [#185](https://gitub.com/greenbone/ospd-openvas/pull/185)
- Set reverse lookup options. [#185](https://github.com/greenbone/ospd-openvas/pull/185)

Not sure what happened here, especially as the link was correct previously.

@jjnicola jjnicola merged commit 3c7bfb8 into greenbone:master Feb 17, 2020
@jjnicola jjnicola deleted the check-feed branch February 17, 2020 07:58
ArnoStiefvater pushed a commit to ArnoStiefvater/ospd-openvas that referenced this pull request Oct 25, 2021
Accept reverse_lookup_only and reverse_lookup_unify target's options.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants