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

fix: improve kerberos checker #2509

Merged
merged 2 commits into from
Jan 26, 2023
Merged

Conversation

ffontaine
Copy link
Contributor

Signed-off-by: Fabrice Fontaine fabrice.fontaine@orange.com

@ffontaine ffontaine force-pushed the fix-kerberos-checker branch from 29b2882 to 08c3fe2 Compare January 9, 2023 13:48
@ffontaine ffontaine force-pushed the fix-kerberos-checker branch from 08c3fe2 to fcdbd4a Compare January 24, 2023 10:17
ffontaine added a commit to ffontaine/cve-bin-tool that referenced this pull request Jan 25, 2023
- Raise an error if one of the other_products is not found in the
  product list
- Do not always append kerberos to all_the_tests (kerberos
  package_test_data will be provided by intel#2509)
- Drop unneeded comment
- Drop gcc, glibc, rust, sqlite, zlib, etc. from other_products if
  needed

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
- Drop mit:kerberos and get_version as NVD NIST has deprecated this
  unusual versioning since January 2020:
  https://nvd.nist.gov/products/cpe/detail/335C9545-32F8-4473-97BD-636F1532525F?namingFormat=2.3&orderBy=CPEURI&keyword=cpe%3A2.3%3Aa%3Amit%3Akerberos%3A5-1.5.1%3A*%3A*%3A*%3A*%3A*%3A*%3A*&status=FINAL%2CDEPRECATED
- Add debian and openwrt test packages

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
@ffontaine ffontaine force-pushed the fix-kerberos-checker branch from fcdbd4a to 186c3f7 Compare January 25, 2023 10:54
@codecov-commenter
Copy link

codecov-commenter commented Jan 25, 2023

Codecov Report

Merging #2509 (fe3484f) into main (e391eb3) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2509      +/-   ##
==========================================
- Coverage   83.04%   83.01%   -0.04%     
==========================================
  Files         600      600              
  Lines        9887     9872      -15     
  Branches     1355     1353       -2     
==========================================
- Hits         8211     8195      -16     
+ Misses       1345     1344       -1     
- Partials      331      333       +2     
Flag Coverage Δ
longtests 82.48% <100.00%> (-0.04%) ⬇️
win-longtests 75.04% <100.00%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
test/test_checkers.py 95.16% <ø> (ø)
test/test_csv2cve.py 100.00% <ø> (ø)
test/test_input_engine.py 100.00% <ø> (ø)
cve_bin_tool/checkers/kerberos.py 100.00% <100.00%> (+9.52%) ⬆️
test/test_data/kerberos.py 100.00% <100.00%> (ø)
cve_bin_tool/output_engine/console.py 95.20% <0.00%> (-2.40%) ⬇️
test/test_scanner.py 75.60% <0.00%> (-0.61%) ⬇️
cve_bin_tool/cli.py 68.16% <0.00%> (+0.32%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

ffontaine added a commit to ffontaine/cve-bin-tool that referenced this pull request Jan 26, 2023
- Raise an error if one of the other_products is not found in the
  product list
- Do not always append kerberos to all_the_tests (kerberos
  package_test_data will be provided by intel#2509)
- Drop gcc, glibc, rust, sqlite, zlib, etc. from other_products if
  needed

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
ffontaine added a commit to ffontaine/cve-bin-tool that referenced this pull request Jan 26, 2023
- Raise an error if one of the other_products is not found in the
  product list
- Do not always append kerberos to all_the_tests (kerberos
  package_test_data will be provided by intel#2509)
- Drop gcc, glibc, rust, sqlite, zlib, etc. from other_products if
  needed

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
terriko pushed a commit that referenced this pull request Jan 26, 2023
- Raise an error if one of the other_products is not found in the
  product list
- Do not always append kerberos to all_the_tests (kerberos
  package_test_data will be provided by #2509)
- Drop gcc, glibc, rust, sqlite, zlib, etc. from other_products if
  needed

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
@terriko
Copy link
Contributor

terriko commented Jan 26, 2023

Updating branch for tests, but I think this should be ok to merge.

@terriko terriko linked an issue Jan 26, 2023 that may be closed by this pull request
Copy link
Contributor

@terriko terriko left a comment

Choose a reason for hiding this comment

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

phew, tests behaving. Thanks for looking into this one and getting it updated!

@terriko terriko merged commit ede655b into intel:main Jan 26, 2023
@ffontaine ffontaine deleted the fix-kerberos-checker branch January 27, 2023 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tests: fix kerberos test data / force running of kerberos tests
3 participants