Skip to content
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

Merge Develop into Release #3500

Open
wants to merge 8 commits into
base: release
Choose a base branch
from
Open

Conversation

r2c-argo[bot]
Copy link
Contributor

@r2c-argo r2c-argo bot commented Oct 25, 2024

Created automatically with the Argo bot using the Argo workflow in release-workflow.yaml

aryx and others added 8 commits September 18, 2024 11:17
* PHP tainted exec

When user input is passed to a function that executes a shell command, without
escaping.

* Correct message string YAML operator

Co-authored-by: Pieter De Cremer (Semgrep) <[email protected]>

---------

Co-authored-by: Pieter De Cremer (Semgrep) <[email protected]>
Co-authored-by: Lewis <[email protected]>
* Upload dockerd socket mount detection rule and test file

* Update dockerd-socket-mount.dockerfile

* Update documentbuilderfactory-disallow-doctype-decl-missing.yaml

Update the rule for checking if FEATURE_SECURE_PROCESSING is set to TRUE for DocumentBuilderFactory object.

* Revert "Update documentbuilderfactory-disallow-doctype-decl-missing.yaml"

This reverts commit c1e2281.

---------

Co-authored-by: Pieter De Cremer (Semgrep) <[email protected]>
* Switch to osemgrep test --experimental

test plan:
wait for green CI check

* comment

* comments
* remove fingerprints/fingerprints.yaml

No idea what this file is, but it's annoying because
we have to skip it in many scripts because it does not
contain regular rules and target test files.
Let's just remove it to simplify things.

test plan:
wait for green CI checks

* remove every use of fingerprints (each time it was to skip the dir)
test plan:
osemgrep test on those dirs do not report any more warnings
about wrong annotations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants