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

Fix whole-only config family selection #1517

Merged

Conversation

timopollmeier
Copy link
Member

@timopollmeier timopollmeier commented May 11, 2021

What:
When modifying the family selection of a scan config, "whole-only" must
be either growing with all VTs or static and empty.
The error message if this is not the case now also includes the name of
the (first) rejected family.

Why:
This selection should ensure that these special families are either selected
completely or not at all and the growing/static ensures they stay that way
if new NVTs are added.

How did you test it:
By modifying a config in GSA to include some of the "whole-only" LSC families
and exclude others.
With greenbone/gsa#2905 the config should be accepted, without the changes
there should be an error message for the first unselected "whole-only" family.

Checklist:

When modifying the family selection of a scan config, "whole-only" must
be either growing with all VTs or static and empty.
The error message if this is not the case now also includes the name of
the (first) rejected family.
@timopollmeier timopollmeier requested a review from a team as a code owner May 11, 2021 09:25
@timopollmeier timopollmeier added the backport-to-main This pull request will be ported to the master branch label May 11, 2021
@nichtsfrei nichtsfrei merged commit f067b61 into greenbone:gvmd-21.04 May 14, 2021
nichtsfrei added a commit that referenced this pull request May 14, 2021
Fix whole-only config family selection (backport #1517)
@jasons1987
Copy link

Hi

Any news on when this might be resolved?

Thanks

@bjoernricks
Copy link
Contributor

It is resolved. The patch is in the 21.04 branch. Are you waiting for a release?

@jasons1987
Copy link

Im running version 21.04.0

Not sure on how to patch this issue?

@bjoernricks
Copy link
Contributor

So I guess you are using some disto packages. In that case we can't do very much for you besides doing a new release https://community.greenbone.net/t/frequently-asked-questions-faq/5558#heading--update-my-gvm-version

@jasons1987
Copy link

So i this resolved on this GVM OS?

When will there be a new release?

Thanks for the info btw

@bjoernricks
Copy link
Contributor

Yes this should be fixed in our GSM products already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-main This pull request will be ported to the master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants