Skip to content

Activate Java Lambda AppSec and API Security system tests - #8128

Merged
claponcet merged 10 commits into
mainfrom
clara.poncet/java-lambda-response-schema-weblog
Oct 27, 2026
Merged

claponcet merged 10 commits into
mainfrom
clara.poncet/java-lambda-response-schema-weblog

Conversation

@claponcet

@claponcet claponcet commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Motivation

Validate the Java Lambda AppSec and API Security features available in tracer v1.67.0 through system tests.

Changes

  • Activate non-blocking WAF, schema collection, fingerprinting, trace tagging, and the 16 previously XPASSing query/RASP checks in the Java Lambda manifest.
  • Replace broad RASP exclusions with specific exclusions so supported tests can run while unsupported blocking and other features remain marked as missing.
  • Update the Java Lambda weblog to return the expected response headers and echo form payloads as JSON for response schema checks.

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes
  3. Mark it as ready for review
    • Tests, manifest, weblog are modified -> you'll need a review from system-tests-reviewers: ask to one of youre co-worker familiar with the tested feature.
    • Framework is modified, or non obvious usage of it -> get a review from system-tests-core (slack)

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

🛟 #apm-shared-testing 🛟

@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

manifests/java_lambda.yml                                               @DataDog/system-tests-reviewers
utils/build/docker/java_lambda/src/main/java/com/datadoghq/lambda/Handler.java  @DataDog/system-tests-reviewers
utils/build/docker/lambda_proxy/alb.py                                  @DataDog/system-tests-reviewers

@datadog-prod-us1-6

datadog-prod-us1-6 Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Pipelines  Tests

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 139ec48 | Docs | View more details | Give us feedback!

@claponcet
claponcet marked this pull request as ready for review October 26, 2026 09:36
@claponcet
claponcet requested a review from a team as a code owner October 26, 2026 09:36
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-26T09:40:04.132910Z 4335108 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@jandro996 jandro996 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! just a few comments

Comment thread manifests/java_lambda.yml
Comment thread manifests/java_lambda.yml Outdated
@claponcet
claponcet requested a review from a team as a code owner October 27, 2026 09:51
@gh-worker-ownership-write-b05516
gh-worker-ownership-write-b05516 Bot removed the request for review from a team October 27, 2026 10:11
@claponcet
claponcet merged commit eea4e87 into main Oct 27, 2026
114 checks passed
@claponcet
claponcet deleted the clara.poncet/java-lambda-response-schema-weblog branch October 27, 2026 10:26
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.

2 participants