Skip to content

Fix: kb items for scannerctl#2167

Merged
Kraemii merged 1 commit intomainfrom
fix-scannerctl-kb
Mar 17, 2026
Merged

Fix: kb items for scannerctl#2167
Kraemii merged 1 commit intomainfrom
fix-scannerctl-kb

Conversation

@Kraemii
Copy link
Member

@Kraemii Kraemii commented Mar 16, 2026

There was a mismatch, when using a hostname instead of an IP address as a target. The kb items were written to the IP address, where as they were read using the original target string.

Here the corresponding KB Context Key: https://github.com/greenbone/openvas-scanner/blob/main/rust/src/scannerctl/interpret/mod.rs#L139

Jira: SC-1576

@Kraemii Kraemii requested a review from a team as a code owner March 16, 2026 11:28
@github-actions
Copy link

github-actions bot commented Mar 16, 2026

Dependency Review

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

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 5027ea6.
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.

Scanned Files

None

@github-actions github-actions bot added the patch_release creates a patch release label Mar 16, 2026
@github-actions github-actions bot added patch_release creates a patch release and removed patch_release creates a patch release labels Mar 16, 2026
There was a missmatch, when using a hostname instead of an IP address as a target. The kb items were written to the IP address, where as they were read using the original target string.
@github-actions github-actions bot added patch_release creates a patch release and removed patch_release creates a patch release labels Mar 17, 2026
@Kraemii Kraemii merged commit 76234c5 into main Mar 17, 2026
35 checks passed
@Kraemii Kraemii deleted the fix-scannerctl-kb branch March 17, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch_release creates a patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants