Skip to content
Snippets Groups Projects

Use scanner instead of analyzer for scan.scanner

Merged Adam Cohen requested to merge use-scanner-instead-of-analyzer into master
All threads resolved!

What does this MR do?

This MR switches the scan.scanner details to use the scanner information instead of the analyzer information. See Security report provide Scanner version, not Analyzer version for details.

"scan": {
  "scanner": {
    "id": "gitleaks",
    "name": "Gitleaks",
    "url": "https://github.com/zricethezav/gitleaks",
    "vendor": {
      "name": "GitLab"
    },
    "version": "5.0.1"
  },
  "type": "secret_detection"
}

What are the relevant issue numbers?

gitlab-org/gitlab#202053 (closed)

Does this MR meet the acceptance criteria?

Edited by Adam Cohen

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Fabien Catteau
  • Fabien Catteau
  • Fabien Catteau
  • Adam Cohen added 1 commit

    added 1 commit

    Compare with previous version

  • Adam Cohen unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Adam Cohen changed title from Use scanner instead of analyzer to Use scanner instead of analyzer for scan.scanner

    changed title from Use scanner instead of analyzer to Use scanner instead of analyzer for scan.scanner

  • Daniel Paul Searles approved this merge request

    approved this merge request

  • Adam Cohen resolved all threads

    resolved all threads

  • Adam Cohen added 9 commits

    added 9 commits

    Compare with previous version

  • Adam Cohen assigned to @dsearles and unassigned @adamcohen

    assigned to @dsearles and unassigned @adamcohen

  • Adam Cohen added 1 commit

    added 1 commit

    Compare with previous version

  • Daniel Paul Searles approved this merge request

    approved this merge request

  • Daniel Paul Searles resolved all threads

    resolved all threads

  • Daniel Paul Searles unapproved this merge request

    unapproved this merge request

  • assigned to @adamcohen and unassigned @dsearles

  • Adam Cohen added 17 commits

    added 17 commits

    Compare with previous version

  • Adam Cohen added 1 commit

    added 1 commit

    • 9538c6a4 - Use stale expectation to get test green

    Compare with previous version

  • Adam Cohen assigned to @dsearles and unassigned @adamcohen

    assigned to @dsearles and unassigned @adamcohen

  • Daniel Paul Searles approved this merge request

    approved this merge request

  • Adam Cohen resolved all threads

    resolved all threads

  • Adam Cohen changed the description

    changed the description

  • Adam Cohen mentioned in commit bd6b6127

    mentioned in commit bd6b6127

  • merged

  • Please register or sign in to reply
    Loading