-
Notifications
You must be signed in to change notification settings - Fork 1.8k
helm charts: merge master into helm-charts branch #14049
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: helm-charts
Are you sure you want to change the base?
helm charts: merge master into helm-charts branch #14049
Conversation
…kflows/test-helm-chart.yml) (DefectDojo#13774) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Bumps [boto3](https://github.com/boto/boto3) from 1.41.1 to 1.41.4. - [Release notes](https://github.com/boto/boto3/releases) - [Commits](boto/boto3@1.41.1...1.41.4) --- updated-dependencies: - dependency-name: boto3 dependency-version: 1.41.4 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…1 (helm/defectdojo/values.yaml) (DefectDojo#13726) * Update nginx/nginx-prometheus-exporter Docker tag from 1.4.2 to v1.5.1 (helm/defectdojo/values.yaml) * update Helm documentation --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: kiblik <5609770+kiblik@users.noreply.github.com>
* feat(helm): Use Valkey Signed-off-by: kiblik <5609770+kiblik@users.noreply.github.com> * apply changes from @fernandezcuesta Signed-off-by: kiblik <5609770+kiblik@users.noreply.github.com> * adjustments Signed-off-by: kiblik <5609770+kiblik@users.noreply.github.com> --------- Signed-off-by: kiblik <5609770+kiblik@users.noreply.github.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…efectDojo#13781) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Bumps [boto3](https://github.com/boto/boto3) from 1.41.4 to 1.41.5. - [Release notes](https://github.com/boto/boto3/releases) - [Commits](boto/boto3@1.41.4...1.41.5) --- updated-dependencies: - dependency-name: boto3 dependency-version: 1.41.5 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: kiblik <5609770+kiblik@users.noreply.github.com>
Remove unnecessary error logging for finding group status.
…hub/workflows/renovate.yaml) (DefectDojo#13788) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…limiting JIRA: add retry/rate limit support
fix: enable uwsgi DD_UWSGI_EXTRA_ARGS passthrough
…-patch-3 Remove left over log statement
Release 2.53.0: Merge Bugfix into Dev
Release: Merge release into master from: release/2.53.0
…efectDojo#13787) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
….53.0-2.54.0-dev Release: Merge back 2.53.0 into dev from: master-into-dev/2.53.0-2.54.0-dev
…x/2.53.0-2.54.0-dev Release: Merge back 2.53.0 into bugfix from: master-into-bugfix/2.53.0-2.54.0-dev
…e.json) (DefectDojo#13792) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
… v2.5.0 (.github/workflows/release-x-manual-helm-chart.yml) (DefectDojo#13793) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Bumps [celery](https://github.com/celery/celery) from 5.5.3 to 5.6.0. - [Release notes](https://github.com/celery/celery/releases) - [Changelog](https://github.com/celery/celery/blob/main/Changelog.rst) - [Commits](celery/celery@v5.5.3...v5.6.0) --- updated-dependencies: - dependency-name: celery dependency-version: 5.6.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
) Bumps [django-pghistory](https://github.com/AmbitionEng/django-pghistory) from 3.8.3 to 3.9.0. - [Release notes](https://github.com/AmbitionEng/django-pghistory/releases) - [Changelog](https://github.com/AmbitionEng/django-pghistory/blob/main/CHANGELOG.md) - [Commits](AmbitionEng/django-pghistory@3.8.3...3.9.0) --- updated-dependencies: - dependency-name: django-pghistory dependency-version: 3.9.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…DefectDojo#13797) Bumps [drf-spectacular-sidecar](https://github.com/tfranzel/drf-spectacular-sidecar) from 2025.10.1 to 2025.12.1. - [Commits](tfranzel/drf-spectacular-sidecar@2025.10.1...2025.12.1) --- updated-dependencies: - dependency-name: drf-spectacular-sidecar dependency-version: 2025.12.1 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Add permission classes and refine queryset in BurpRawRequestResponseViewSet
docs: add new page on custom trust
[docs] Pro changelog: Dec 30
[docs] create sitemap at root on Hugo deploy
Release 2.54.0: Merge Bugfix into Dev
Release: Merge release into master from: release/2.54.0
|
wdyt @kiblik |
🔴 Risk threshold exceeded.This pull request contains multiple detections: repeated sensitive edits to docker/entrypoint-uwsgi.sh and critical workflow injection issues where unvalidated workflow_dispatch inputs are interpolated into shell and actions/github-script steps (in .github/workflows/release-1-create-pr.yml and release-3-master-into-dev.yml), enabling arbitrary shell/JavaScript code execution and potential token exfiltration. Immediate remediation: restrict/validate inputs, sanitize interpolation, and lock down sensitive file changes in .dryrunsecurity.yaml.
🔴 Configured Codepaths Edit in
|
| Vulnerability | Configured Codepaths Edit |
|---|---|
| Description | Sensitive edits detected for this file. Sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. |
🔴 Configured Codepaths Edit in docker/entrypoint-uwsgi.sh
| Vulnerability | Configured Codepaths Edit |
|---|---|
| Description | Sensitive edits detected for this file. Sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. |
🔴 Configured Codepaths Edit in docker/entrypoint-uwsgi.sh
| Vulnerability | Configured Codepaths Edit |
|---|---|
| Description | Sensitive edits detected for this file. Sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. |
JavaScript Injection in 'actions/github-script' in .github/workflows/release-1-create-pr.yml
| Vulnerability | JavaScript Injection in 'actions/github-script' |
|---|---|
| Description | The 'NEW_BRANCH' environment variable is interpolated directly into a JavaScript string within an 'actions/github-script' block. This variable is derived from the 'release_number' input provided via 'workflow_dispatch'. While there are validation steps using 'grep' to check the format of 'release_number', this validation is insufficient because 'grep' returns a successful exit code if any line in a multiline input matches the pattern. An attacker can provide a multiline input where the first line passes validation and subsequent lines inject a malicious value for 'NEW_BRANCH' into the GITHUB_ENV file. When this value is interpolated into the JavaScript code, it allows for arbitrary code execution in the context of the runner, which has access to the GITHUB_TOKEN. |
django-DefectDojo/.github/workflows/release-1-create-pr.yml
Lines 117 to 118 in a69d0fe
| title: 'Release: Merge release into master from: ${{ env.NEW_BRANCH }}', | |
| body: `Release triggered by \`${ process.env.GITHUB_ACTOR }\``, |
Shell and JavaScript Injection via Release Inputs in .github/workflows/release-3-master-into-dev.yml
| Vulnerability | Shell and JavaScript Injection via Release Inputs |
|---|---|
| Description | The workflow 'release-3-master-into-dev.yml' uses 'workflow_dispatch' inputs 'release_number_new' and 'release_number_dev' directly in shell 'run' steps and 'github-script' blocks without any validation or sanitization. This allows an attacker with permission to trigger the workflow to execute arbitrary shell commands or JavaScript code in the context of the GitHub Actions runner. For example, providing a malicious string like 1.0.0"; touch /tmp/pwned; # as an input can lead to command execution. Similarly, in the 'github-script' step, inputs are interpolated into a JavaScript object literal, allowing for code injection that can exfiltrate secrets or perform unauthorized actions using the provided GITHUB_TOKEN. |
django-DefectDojo/.github/workflows/release-3-master-into-dev.yml
Lines 49 to 52 in a69d0fe
| sed -ri 's/__version__ = ".*"/__version__ = "${{ inputs.release_number_dev }}"/' dojo/__init__.py | |
| sed -ri 's/"version": ".*"/"version": "${{ inputs.release_number_dev }}"/' components/package.json | |
| sed -ri 's/appVersion: ".*"/appVersion: "${{ inputs.release_number_dev }}"/' helm/defectdojo/Chart.yaml | |
| CURRENT_CHART_VERSION=$(grep -oP 'version: (\K\S*)?' helm/defectdojo/Chart.yaml | head -1) |
We've notified @mtesauro.
All finding details can be found in the DryRun Security Dashboard.
This PR merges the
masterbranch into thehelm-chartswhile making sure the helm chart files are all preserved including their history/commits.The reason for this PR being so big is that the
helm-chartdiverted frommasterin 2017. I think it's good to do this merge to make thehelm-chartsbranch more in sync withmaster. Maybe at some point we need to add a step to the release process to this "merge back" after every release (unless we split off the helm-chart into its own repo).With this branch checked out the helm chart files look OK, as well as the history. Also the github workflows now have the current content, which is maybe the prime reason for this PR.
This PR comes intead of #13425 which at the time of writing cannot be viewed on github.com. Maybe it's to big/complex or has too many conflicts?