Skip to content

added some additional color on creating web hooks for slack to help the next guy. Also adjusted two typos#770

Closed
nobletrout wants to merge 44 commits intosecureCodeBox:mainfrom
nobletrout:main
Closed

added some additional color on creating web hooks for slack to help the next guy. Also adjusted two typos#770
nobletrout wants to merge 44 commits intosecureCodeBox:mainfrom
nobletrout:main

Conversation

@nobletrout
Copy link
Copy Markdown
Contributor

Description

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure npm test runs for the whole project.
  • Make codeclimate checks happy

next guy. Also adjusted two typos

Signed-off-by: nobletrout <nobletrout@gmail.com>
…to main

Signed-off-by: nobletrout <nobletrout@gmail.com>
rfelber
rfelber previously approved these changes Oct 24, 2021
Copy link
Copy Markdown
Member

@rfelber rfelber left a comment

Choose a reason for hiding this comment

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

Hi @nobletrout,

thx a lot for your PR and improvement here regarding the project docs 🤙 Great work!

Do you mind to add yourself to our CONTRIBUTORS File to agree with our project licence apache 2.0 and ensure all of your commits are signed off? Thanks in advance 🤗

nobletrout and others added 23 commits October 25, 2021 09:47
next guy. Also adjusted two typos

Signed-off-by: nobletrout <nobletrout@gmail.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
The old version of the integration test framework did not have support
for the new initContainer syntax introduced in ADR-0009. This commit
adds this support, which is necessary for the semgrep integration tests

Signed-off-by: Max Maass <max.maass@iteratec.com>
These tests require network access to download test files, as I do not
know of a good way to provision the test files for them. Alternatives
are appreciated.

Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
We do not want to save the matched lines into the s3 bucket without
encryption, as the lines may include sensitive information.

Signed-off-by: Max Maass <max.maass@iteratec.com>
The semgrep scanner will not ship with default cascadingRules, so we
provide documentation on how to write your own instead.

Signed-off-by: Max Maass <max.maass@iteratec.com>
We do not want to retrieve files from the Internet for the integration
tests, so the test now uses a local file in the repo.

Signed-off-by: Max Maass <max.maass@iteratec.com>
DefectDojo supports semgrep, so this commit adds support for the semgrep
results to the DD hook. It works fine on individual scans, but I still
want to do some more testing before merging this.

Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
malexmave and others added 16 commits October 25, 2021 09:47
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: secureCodeBoxBot <securecodebox@iteratec.com>
Signed-off-by: secureCodeBoxBot <securecodebox@iteratec.com>
Signed-off-by: Robert Seedorff <Robert.Seedorff@iteratec.com>
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
Signed-off-by: secureCodeBoxBot <securecodebox@iteratec.com>
Signed-off-by: nobletrout <nobletrout@gmail.com>
@malexmave
Copy link
Copy Markdown
Member

I think this PR was superceeded with #781, which has now been merged, so I'll close this one. Let me know if I got something wrong.

@malexmave malexmave closed this Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants