FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add Wpscan wrapper by the-simmon · Pull Request #1260 · secureCodeBox/secureCodeBox · GitHub

/ secureCodeBox Public

Add Wpscan wrapper - #1260

Merged
J12934 merged 13 commits into
mainfrom
wpscan_wrapper
Jul 12, 2022
ConversationCommits (13)ChecksFiles changed
Merged

Add Wpscan wrapper#1260
J12934 merged 13 commits into
mainfrom
wpscan_wrapper

Conversation

the-simmon commented Jul 5, 2022 •
edited by rfelber
Loading

Copy link
Copy Markdown
Contributor

Solves #1083

Adds a wrapper to wpscan because wpscan does return non 0 values when it find vulnerability's.

Add wrapper to wpscan
700bcf9 
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Add explanation for wrapper
63a85e8 
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Pass through exit code on error
526f6a6 
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
the-simmon added bug Bugs scanner Implement or update a security scanner planned Issues we will do in the next sprint. labels Jul 5, 2022
the-simmon self-assigned this Jul 5, 2022
the-simmon and others added 3 commits July 5, 2022 11:35
Updating Helm Docs
ffe7385 
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
Add exit code 4 to if
0070f04 
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Merge branch 'wpscan_wrapper' of https://github.com/secureCodeBox/sec…
db9dbe9 
…ureCodeBox into wpscan_wrapper

github-actions Bot commented Jul 5, 2022 •
edited
Loading

Copy link
Copy Markdown

MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
⚠️ ACTION actionlint 2 15 0.06s
⚠️ BASH bash-exec 1 1 0.0s
⚠️ BASH shellcheck 1 2 0.01s
⚠️ BASH shfmt 1 1 0.0s
✅ DOCKERFILE dockerfilelint 1 0 0.33s
✅ DOCKERFILE hadolint 1 0 0.07s
✅ GIT git_diff yes no 0.16s
✅ GROOVY npm-groovy-lint 1 0 7.01s
⚠️ JAVA checkstyle 1 1 2.06s
⚠️ JAVASCRIPT eslint 1 1 0.74s
✅ JSON eslint-plugin-jsonc 2 0 1.27s
✅ JSON jsonlint 2 0 0.85s
⚠️ JSON prettier 2 1 0.67s
✅ JSON v8r 2 0 4.24s
✅ SPELL misspell 15 0 0.04s
⚠️ YAML prettier 7 1 0.78s
⚠️ YAML v8r 7 1 5.51s
⚠️ YAML yamllint 7 1 0.33s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

SebieF reviewed Jul 5, 2022
View reviewed changes

SebieF left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Only one small suggestion, rest looks good to me :)

Comment thread scanners/wpscan/scanner/wrapper.sh Outdated
the-simmon and others added 3 commits July 5, 2022 19:05
Refactor if statement
f7f2c7a 
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Add new docker image to values.yaml
20be08c 
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Updating Helm Docs
03b59e6 
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>

J12934 commented Jul 6, 2022

Copy link
Copy Markdown
Member

Hey @SebieF

Well also need to add wpscan to the list of scanners to build a custom image for in the release build pipeline:

secureCodeBox/.github/workflows/release-build.yaml

Line 362 in a6eae05

scanner:

Adding wpscan to third party scanner build
def7734 
Signed-off-by: Sebastian Franz <sebastian.franz@iteratec.com>
rfelber previously requested changes Jul 9, 2022
View reviewed changes

rfelber left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

wpscan-scan-type.yaml:

name: "wpscan{{ .Values.scanner.nameAppend | default ""}}"

There are to many " chars in one string 🤔

Comment thread scanners/wpscan/values.yaml Outdated
the-simmon and others added 3 commits July 9, 2022 22:13
Fix typo
706aecc 
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Merge branch 'wpscan_wrapper' of https://github.com/secureCodeBox/sec…
89e96d2 
…ureCodeBox into wpscan_wrapper
Updating Helm Docs
ff9c702 
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
J12934 merged commit dae41cd into main Jul 12, 2022
J12934 deleted the wpscan_wrapper branch July 12, 2022 11:11
This was referenced Nov 30, 2023
Open
Open
Open
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

SebieF SebieF left review comments
J12934 J12934 approved these changes
rfelber rfelber left review comments

Assignees

the-simmon

Labels

bug Bugs planned Issues we will do in the next sprint. scanner Implement or update a security scanner

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL