← 输入框|github.com|刷新
Skip to content

Change host to hostname in zap parser - #1346

Merged
Ilyesbdlala merged 4 commits into
mainfrom
maintenance/zap-rename-to-hostname
Nov 1, 2022
Merged

Ilyesbdlala merged 4 commits into
mainfrom
maintenance/zap-rename-to-hostname

Conversation

@the-simmon

Copy link
Copy Markdown
Contributor

This PR solves #853 partly, in particular this comment

@the-simmon the-simmon added scanner Implement or update a security scanner maintenance breaking Changes requiring a major release labels Sep 7, 2022
@the-simmon the-simmon self-assigned this Sep 7, 2022
@github-actions

github-actions Bot commented Sep 7, 2022 •

Copy link
Copy Markdown

MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
⚠️ JAVASCRIPT eslint 1 1 1.07s
✅ SPELL misspell 6 0 2.47s
✅ YAML prettier 5 0 4.26s

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

@malexmave malexmave added this to the v4.0.0 milestone Sep 7, 2022

@malexmave malexmave left a comment

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.

Don't forget to also update the example findings for ZAP and ZAP advanced (they are shown on the documentation site) :)

@Ilyesbdlala Ilyesbdlala added breaking Changes requiring a major release and removed breaking Changes requiring a major release labels Oct 6, 2022
@secureCodeBoxBot

Copy link
Copy Markdown
Contributor

This pull request includes breaking changes. Please make sure that you included the breaking changes and the steps required to upgrade in UPGRADING.md.
✨ Thank you for your contribution! ✨

@the-simmon
the-simmon force-pushed the maintenance/zap-rename-to-hostname branch from 4766072 to 31afea9 Compare October 12, 2022 09:31
@J12934
J12934 dismissed malexmave’s stale review October 13, 2022 11:12

resolved 😀

J12934
J12934 previously approved these changes Oct 13, 2022
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>

@Ilyesbdlala Ilyesbdlala left a comment

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.

LGTM.

@Ilyesbdlala
Ilyesbdlala merged commit fb3c4fb into main Nov 1, 2022
@Ilyesbdlala
Ilyesbdlala deleted the maintenance/zap-rename-to-hostname branch November 1, 2022 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Changes requiring a major release hacktoberfest-accepted maintenance scanner Implement or update a security scanner

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants