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

Perform a scan even if there are missing plugins. #439

Merged
merged 2 commits into from
Dec 3, 2019

Conversation

jjnicola
Copy link
Member

@jjnicola jjnicola commented Dec 2, 2019

No description provided.

@jjnicola jjnicola marked this pull request as ready for review December 2, 2019 15:32
@bjoernricks
Copy link
Contributor

Would it be possible to raise an error result in this case too? It would be good to display an error in the report for this issue. not only in the logs of the machine.

src/attack.c Outdated Show resolved Hide resolved
This allows to detect errors in attack.c and send a message to client.
Copy link
Contributor

@bjoernricks bjoernricks left a comment

Choose a reason for hiding this comment

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

looks good to me

Copy link
Contributor

@bjoernricks bjoernricks left a comment

Choose a reason for hiding this comment

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

Looks good to me

@jjnicola jjnicola merged commit 71f4b5e into greenbone:openvas-7.0 Dec 3, 2019
@jjnicola jjnicola deleted the nvti-failure-7 branch December 3, 2019 13:20
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.

2 participants