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

Extend command line options for managing scanners (master) #866

Merged

Conversation

timopollmeier
Copy link
Member

@timopollmeier timopollmeier commented Nov 14, 2019

This is a port of #815, which extends the command line options --get-scanners, --create-scanner and --modify-scanner to provide more information and accept more scanner types.

Checklist:

The option now also returns the type, host and port of each scanner.
This adds the scanner types "GMP" and "OSP-Sensor" to these options as
well as the option to use the numeric type value used in GMP.
Error messages when using the options are now always sent to stderr and
are more specific about credential issues.
The new types and use of the numeric value are added to the --help
output and man page.
This allows setting the credential of a scanner to an existing one or to
unset the credential.
Instead of numbers the command will now return the types as strings as
accepted by --create-scanner and --modify-sensor.
@timopollmeier timopollmeier marked this pull request as ready for review November 14, 2019 15:10
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.

Currently I am not sure about the new sensor type name. We need to discuss it before merging this pr.

@timopollmeier
Copy link
Member Author

@bjoernricks The name is already used internally and in the 9.0 branch, but it's not exposed to the user yet in the released version (9.0.0).
If we want to change the name, maybe we could also create new PRs to change this consistently.

@bjoernricks
Copy link
Contributor

@timopollmeier i really would like to use the term Greenbone-Sensor. IMHO OSP is only an implementation detail.

@bjoernricks
Copy link
Contributor

But I am fine with changing this in another PR.

@bjoernricks bjoernricks self-requested a review December 10, 2019 08:13
@timopollmeier timopollmeier merged commit 4c075cb into greenbone:master Dec 10, 2019
@timopollmeier timopollmeier deleted the extend-scanners-cli-master branch June 9, 2020 06:54
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