Skip to content

Feat: Reclassify Confidence as Severity

Lucas Charles requested to merge reclassify-confidence-as-severity into master

What does this MR do?

Per gitlab-org&4004 (comment 424620928), we should remap brakeman's reported confidence field to severity.

We previously reported the brakeman level as confidence, however according to the docs, brakeman conflates confidence and severity so we should map both accordingly:

See https://github.com/presidentbeef/brakeman/issues/1502#issuecomment-674208184 and https://github.com/presidentbeef/brakeman#confidence-levels

In contrast to a similar change for flawfinder, the inflation of both confidence and severity implies to me that we should be remapping brakeman's confidence level to both our confidence and severity, where-as flawfinder was previously a mistake before.

What are the relevant issue numbers?

gitlab-org&4004 (closed)

Does this MR meet the acceptance criteria?

Release post

gitlab-com/www-gitlab-com!67951 (merged)

Edited by Taylor McCaslin

Merge request reports

Loading