Skip to content

Support setting env variables for Scans generated by the Kubernetes AutoDiscovery#2628

Merged
J12934 merged 3 commits intosecureCodeBox:mainfrom
J12934:autodiscovery-support-setting-env-for-generated-scans
Aug 27, 2024
Merged

Support setting env variables for Scans generated by the Kubernetes AutoDiscovery#2628
J12934 merged 3 commits intosecureCodeBox:mainfrom
J12934:autodiscovery-support-setting-env-for-generated-scans

Conversation

@J12934
Copy link
Copy Markdown
Member

@J12934 J12934 commented Aug 23, 2024

Description

This allows to set the env vars on the Scan.
This can be helpful when you want to configure some part of the scan via the environment vars.
In particular I'd like to have this so that i can use the zap-automation-framework with a generic config and overwrite the target based on the service discovered by the auto-discovery.

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

J12934 added 3 commits August 23, 2024 11:55
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
@J12934 J12934 added enhancement New feature or request auto-discovery labels Aug 23, 2024
@J12934 J12934 self-assigned this Aug 23, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Aug 23, 2024

Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit 04e2f9a
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/66c85d84f05cac0008336113

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@Reet00 Reet00 left a comment

Choose a reason for hiding this comment

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

Looks good. Test is also fine

@J12934 J12934 merged commit a974b89 into secureCodeBox:main Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-discovery enhancement New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants