Skip to content

Check feature flag when deciding occurrence state

rossfuhrman requested to merge check-flag-for-occurrence-state into master

What does this MR do?

In support of First-class vulnerabilities backend MVC implementation, the state method was added to the Vulnerabilities::Occurrence model. 4c381288

This MR adds a check of :first_class_vulnerabilities feature flag to prevent unnecessary attempts to load vulnerabilities to determine the state of the occurrence.

There is already an outstanding issue to address the performance when the feature flag is set to true: #34003 (closed)

Screenshots

Screenshot of unnecessary Vulnerability Load calls that this MR avoids when the :first_class_vulnerabilities feature flag is off. Screen_Shot_2020-01-10_at_9.51.38_AM

Does this MR meet the acceptance criteria?

Conformity

Edited by rossfuhrman

Merge request reports