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

Add option for "Start task" on "new secinfo arrived" for alerts #2419

Merged
merged 2 commits into from
Aug 24, 2020

Conversation

swaterkamp
Copy link
Member

What:
There is the new alerts feature of starting a task upon the arrival of new secinfo. gvmd has it implemented in greenbone/gvmd#957 and greenbone/gvmd#960. This PR adds the start task option to the corresponding method selection.

Why:
Without these changes, the new feature would not be available in GSA.

How:
The "Start Task" event is pushed into the list of possible methods. From this list I chose the 'Start task' together with the secinfo event condition and clicked on "test alert". The corresponding task started.

Checklist:

@swaterkamp swaterkamp requested review from saberlynx and a team August 24, 2020 13:21
@swaterkamp swaterkamp self-assigned this Aug 24, 2020
@codecov
Copy link

codecov bot commented Aug 24, 2020

Codecov Report

Merging #2419 into gsa-20.08 will increase coverage by 0.16%.
The diff coverage is 82.03%.

Impacted file tree graph

@@              Coverage Diff              @@
##           gsa-20.08    #2419      +/-   ##
=============================================
+ Coverage      52.76%   52.93%   +0.16%     
=============================================
  Files           1070     1070              
  Lines          25789    25831      +42     
  Branches        7312     7358      +46     
=============================================
+ Hits           13607    13673      +66     
+ Misses         11058    11039      -19     
+ Partials        1124     1119       -5     
Impacted Files Coverage Δ
gsa/src/gmp/commands/wizard.js 9.23% <0.00%> (ø)
gsa/src/web/components/error/errorpanel.js 100.00% <ø> (ø)
gsa/src/web/pages/alerts/dialog.js 20.33% <ø> (ø)
gsa/src/web/pages/cpes/details.js 11.11% <ø> (ø)
gsa/src/web/pages/extras/trashactions.js 5.49% <0.00%> (ø)
gsa/src/web/pages/hosts/dialog.js 50.00% <ø> (ø)
gsa/src/web/pages/policies/detailspage.js 100.00% <ø> (ø)
gsa/src/web/pages/policies/row.js 100.00% <ø> (ø)
gsa/src/web/pages/portlists/detailspage.js 34.48% <ø> (ø)
gsa/src/web/pages/portlists/row.js 55.55% <ø> (ø)
... and 37 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b17228...98ce261. Read the comment docs.

@saberlynx saberlynx merged commit 8fad03c into greenbone:gsa-20.08 Aug 24, 2020
@swaterkamp swaterkamp deleted the startTaskOnNewSecInfo branch April 20, 2021 08:48
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.

3 participants