Skip to content

updated security scan to have parity with consul-dataplane#1965

Merged
wilkermichael merged 1 commit intomainfrom
net-2420-update-security-scan
Mar 1, 2023
Merged

updated security scan to have parity with consul-dataplane#1965
wilkermichael merged 1 commit intomainfrom
net-2420-update-security-scan

Conversation

@wilkermichael
Copy link
Copy Markdown
Contributor

Changes proposed in this PR:

How I've tested this PR:
👀

How I expect reviewers to test this PR:
👀

Checklist:

  • [n/a] Tests added
  • [n/a] CHANGELOG entry added

    HashiCorp engineers only, community PRs should not add a changelog entry.
    Entries should use present tense (e.g. Add support for...)

@wilkermichael wilkermichael added backport/0.49.x backport/1.1.x Backport to release/1.1.x branch pr/no-changelog PR does not need a corresponding .changelog entry labels Feb 28, 2023
@wilkermichael wilkermichael requested review from a team, david-yu, erichaberkorn and malizz and removed request for a team February 28, 2023 23:55
@wilkermichael wilkermichael marked this pull request as ready for review March 1, 2023 20:04
@curtbushko curtbushko self-requested a review March 1, 2023 20:21
Copy link
Copy Markdown
Contributor

@curtbushko curtbushko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will probably result in several updates needed. You can run the scanner locally if needed. The release process doc has a link to the scanner repo.

@wilkermichael
Copy link
Copy Markdown
Contributor Author

There is a known issue with the test that is failing, I've fixed it in another PR

@wilkermichael
Copy link
Copy Markdown
Contributor Author

So merging now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/1.1.x Backport to release/1.1.x branch pr/no-changelog PR does not need a corresponding .changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants