fix: Use kubesec rule ID for primary identifier value
What does this MR do?
Changes previous identifier which relied on selectors and rule
descriptions. This is overly specific to the finding, not the rule
itself. Additionally, the Reason
can be blank for some reason with
certain rules, so we should rely on the ID
attribute instead,
falling back to Reason
for backwards compatibility
What are the relevant issue numbers?
Relates to gitlab-org/gitlab#323221 (closed)
Does this MR meet the acceptance criteria?
-
Changelog entry added -
Documentation created/updated for GitLab EE, if necessary -
Documentation created/updated for this project, if necessary -
Documentation reviewed by technical writer or follow-up review issue created -
Tests added for this feature/bug -
Job definition updated, if necessary -
Conforms to the code review guidelines -
Conforms to the Go guidelines -
Security reports checked/validated by reviewer
Edited by Zach Rice