Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support GMP version 22.6 #1196

Merged
merged 6 commits into from
Feb 3, 2025
Merged

Support GMP version 22.6 #1196

merged 6 commits into from
Feb 3, 2025

Conversation

bjoernricks
Copy link
Contributor

@bjoernricks bjoernricks commented Jan 27, 2025

What

Support GMP version 22.6

Why

Implement support for GMP version 22.6 which adds audit reports, report configs and changes get feeds.

References

https://jira.greenbone.net/browse/GEA-868
Closed #1195

Checklist

  • Tests

Also tested some report config request against gvmd manually.

Copy link

github-actions bot commented Jan 27, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 3abd3ae.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA d3fd59e.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link

github-actions bot commented Jan 27, 2025

Conventional Commits Report

Type Number
Added 2

🚀 Conventional commits found.

Copy link

codecov bot commented Jan 27, 2025

Codecov Report

Attention: Patch coverage is 99.29577% with 3 lines in your changes missing coverage. Please review.

Project coverage is 97.74%. Comparing base (8f7299c) to head (3abd3ae).
Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
gvm/protocols/gmp/_gmp.py 77.77% 1 Missing and 1 partial ⚠️
gvm/protocols/gmp/_gmp226.py 97.72% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1196      +/-   ##
==========================================
+ Coverage   97.60%   97.74%   +0.14%     
==========================================
  Files          65       71       +6     
  Lines        4544     4967     +423     
  Branches      825      895      +70     
==========================================
+ Hits         4435     4855     +420     
- Misses         74       76       +2     
- Partials       35       36       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bjoernricks bjoernricks marked this pull request as ready for review January 28, 2025 14:50
@bjoernricks bjoernricks requested a review from a team January 28, 2025 14:50
@bjoernricks bjoernricks enabled auto-merge (rebase) January 28, 2025 15:08
Implement support for GMP version 22.6 which adds audit reports, report
configs and changes get feeds.
The version is already encoded into the module name. For implementing a
new protocol version it's easier when the version in not included in all
test class names.
Not every XML element will have a value/text. Therefore it should be
possible to set it optionally.
Extend GMP classed to support all report config commands.
@bjoernricks bjoernricks merged commit 8fd3d4e into main Feb 3, 2025
21 checks passed
@bjoernricks bjoernricks deleted the support-gmp-v22.6 branch February 3, 2025 08:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GMP Version 22.6 not supported
2 participants