Closed samcornwell closed 4 years ago
Recently we updated the InspecUtils#string_to_impact
and InspecUtils#float_to_impact
methods and this did not come up as a consideration.
If it comes back up we can take a look at the "Do something else" portion because we ported over the InspecUtils#get_impact_string
method and can manipulate what is returned in certain cases more easily.
Right now we are leaving out the
Severity
blank in Checklist files if the controlimpact
is 0. In writing controls, we change the originalimpact
to 0 if it isNot Applicable
. We should re-evaluate if we want to try to parse out the original control severity forNot Applicable
controls, do something else, or leave theSeverity
blank.