Skip to content

Conversation

@christophe-papazian
Copy link
Contributor

@christophe-papazian christophe-papazian commented Dec 29, 2025

Motivation

second part of the move after #5888

Changes

remove iast decorator related to python and migrate them to the manifest (when useful).

APPSEC-60258

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • Anything but tests/ or manifests/ is modified ? I have the approval from R&P team
  • A docker base image is modified?
    • the relevant build-XXX-image label is present
  • A scenario is added, removed or renamed?

@github-actions
Copy link
Contributor

CODEOWNERS have been resolved as:

manifests/python.yml                                                    @DataDog/apm-python @DataDog/asm-python
tests/appsec/iast/sink/test_command_injection.py                        @DataDog/asm-libraries @DataDog/system-tests-core
tests/appsec/iast/sink/test_insecure_cookie.py                          @DataDog/asm-libraries @DataDog/system-tests-core
tests/appsec/iast/sink/test_no_httponly_cookie.py                       @DataDog/asm-libraries @DataDog/system-tests-core
tests/appsec/iast/sink/test_no_samesite_cookie.py                       @DataDog/asm-libraries @DataDog/system-tests-core
tests/appsec/iast/sink/test_nosql_mongodb_injection.py                  @DataDog/asm-libraries @DataDog/system-tests-core
tests/appsec/iast/sink/test_stacktrace_leak.py                          @DataDog/asm-libraries @DataDog/system-tests-core
tests/appsec/iast/sink/test_unvalidated_redirect.py                     @DataDog/asm-libraries @DataDog/system-tests-core
tests/appsec/iast/source/test_parameter_value.py                        @DataDog/asm-libraries @DataDog/system-tests-core
tests/appsec/iast/test_security_controls.py                             @DataDog/asm-libraries @DataDog/system-tests-core

@christophe-papazian christophe-papazian marked this pull request as ready for review December 29, 2025 14:18
@christophe-papazian christophe-papazian requested review from a team as code owners December 29, 2025 14:18
@christophe-papazian christophe-papazian requested review from Yun-Kim and emmettbutler and removed request for a team December 29, 2025 14:18
@christophe-papazian christophe-papazian enabled auto-merge (squash) December 29, 2025 14:25
@christophe-papazian christophe-papazian merged commit d53050f into main Dec 29, 2025
386 checks passed
@christophe-papazian christophe-papazian deleted the christophe-papazian/new_syntaxe_for_iast_python_tests branch December 29, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants