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

[VIRTS-2188] Change Rule action to a dropdown selection instead of textbox #2157

Merged
merged 4 commits into from
May 17, 2021

Conversation

emmanvg
Copy link
Contributor

@emmanvg emmanvg commented May 17, 2021

Description

Make "action:" under the rules modal a dropdown with the only valid options. In the past this would be a textbox that would allow any value, but in reality there are only two valid values ALLOW or DENY. Hence, it makes sense to simplify this form.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Go fact sources, click view sources, click add rule. Type a trait, a match and select a value from the dropdown modal ALLOW or DENY. Check under data/sources/UUID.yml that an entry exists with the proper ALLOW or DENY value. Re-open fact sources, load the test fact source and view the rules, check that nothing has changed.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

@codecov
Copy link

codecov bot commented May 17, 2021

Codecov Report

Merging #2157 (b318593) into master (394852a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2157   +/-   ##
=======================================
  Coverage   66.43%   66.43%           
=======================================
  Files          70       70           
  Lines        5225     5225           
=======================================
  Hits         3471     3471           
  Misses       1754     1754           

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 394852a...b318593. Read the comment docs.

Copy link
Contributor

@clenk clenk left a comment

Choose a reason for hiding this comment

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

LGTM; this has the exact same changes as #2144 which was approved by me and @b17zr.

Copy link
Contributor

@mrengstrom mrengstrom 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

@clenk clenk merged commit 515b4ba into master May 17, 2021
@clenk clenk deleted the emmanvg/virts-2188/make-action-dropdown branch May 17, 2021 14:33
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