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

Adding missing zap-advanced scripts again (closes #656) by rfelber · Pull Request #658 · secureCodeBox/secureCodeBox · GitHub

Adding missing zap-advanced scripts again (closes #656) - #658

Merged
rfelber merged 1 commit into
mainfrom
bugfix/add-missing-zap-advanced-scripts
Sep 20, 2021
Merged

Adding missing zap-advanced scripts again (closes #656)#658
rfelber merged 1 commit into
mainfrom
bugfix/add-missing-zap-advanced-scripts

Conversation

rfelber commented Sep 20, 2021
edited
Loading

Copy link
Copy Markdown
Member

Description

This PR if applied adds the missing zap-advanced scripts again and closes #656

Additional this PR bugfixes a problem within the zap authentication script configuration (if you try to use the scb-oidc-password-grand-type.js script).

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

rfelber added bug Bugs scanner Implement or update a security scanner labels Sep 20, 2021
rfelber self-assigned this Sep 20, 2021
Bugfixing zap authentication script.

Signed-off-by: Robert Seedorff <Robert.Seedorff@iteratec.com>
rfelber force-pushed the bugfix/add-missing-zap-advanced-scripts branch from 8fd7dd6 to e726afa Compare September 20, 2021 19:49
rfelber enabled auto-merge September 20, 2021 19:50
rfelber requested a review from J12934 September 20, 2021 19:51
rfelber merged commit 6ca5a37 into main Sep 20, 2021
rfelber deleted the bugfix/add-missing-zap-advanced-scripts branch September 20, 2021 21:17
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bugs scanner Implement or update a security scanner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

zap-advanced-scripts not in helm chart 3.1.0 release

2 participants


Back | FazBrowse Home | New Git URL