Skip to content

test: Add CVE mapping test for GnuTLS 3.8.0 #4998

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

Merged
merged 6 commits into from
May 5, 2025

Conversation

tutou2356
Copy link
Contributor

@tutou2356 tutou2356 commented Apr 3, 2025

1.This PR adds a new CVE mapping test for GnuTLS 3.8.0 in test/test_data/gnutls.py, as part of the "Adding new tests: CVE mapping tests" task. The new test case includes:

{
    "product": "gnutls",
    "version": "3.8.0",
    "version_strings": ["gnutls-cli 3.8.0", "gnutls-serv 3.8.0", "GnuTLS 3.8.0"]
}
2.I ran the tests, all tests passed, including the new gnutls-3.8.0 test case. Here is the test result screenshot:
![test-pass-picture](https://github.com/user-attachments/assets/68711fd8-8b3d-4e71-bbc6-7a0adbebfdef)

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@tutou2356 tutou2356 force-pushed the add-new-test-case branch from 3477790 to c6a3cb2 Compare April 4, 2025 01:40
@terriko
Copy link
Contributor

terriko commented May 5, 2025

The issue with the other test has been resolved so I'm updating this branch to see if this works, but I forgot to edit the title first so it'll fail gitlint.

@terriko terriko changed the title Add CVE mapping test for GnuTLS 3.8.0 test: Add CVE mapping test for GnuTLS 3.8.0 May 5, 2025
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.

Okay, looks like we're good to merge now. Thank you!

@terriko terriko merged commit 6f225ee into intel:main May 5, 2025
31 of 32 checks passed
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.

None yet

2 participants