Skip to content

Add Java Lambda SCA system-test coverage - #8182

Merged
claponcet merged 17 commits into
mainfrom
clara.poncet/java-lambda-sca
Oct 29, 2026
Merged

claponcet merged 17 commits into
mainfrom
clara.poncet/java-lambda-sca

Conversation

@claponcet

@claponcet claponcet commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Motivation

Add Java Lambda SCA system-test coverage for APPSEC-71306.

Stacked on #8128 (clara.poncet/java-lambda-response-schema-weblog). Related Java agent smoke coverage: DataDog/dd-trace-java#13059.

Changes

  • Add APPSEC_LAMBDA_SCA, reuse the five existing SCA reachability tests, and wire the scenario into the end-to-end workflow and CI change selection.
  • Add the vulnerable-call endpoints and Junrar 7.5.5 to the Java Lambda weblogs. Keep Junrar in a separate JAR so runtime dependency resolution can identify it.
  • Enable the tests for Java Lambda from 1.68.0-SNAPSHOT and check endpoint responses before telemetry assertions.

Validated on the API Gateway v2 emulator (java-apigw-http) with the local Java tracer 1.68.0-SNAPSHOT: all five SCA tests passed. ./format.sh also passed. The other Java Lambda event variants have not been run locally.

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 🛟

@claponcet claponcet added java Pull requests that update Java code ai-generated The pull request includes a significant amount of AI-generated code labels Oct 26, 2026
@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

utils/build/docker/java_lambda/src/main/java/com/datadoghq/lambda/ScaReachability.java  @DataDog/system-tests-reviewers
.github/workflows/run-end-to-end.yml                                    @DataDog/system-tests-core
docs/understand/weblogs/end-to-end_weblog.md                            @DataDog/system-tests-core
manifests/java_lambda.yml                                               @DataDog/system-tests-reviewers
manifests/nodejs_lambda.yml                                             @DataDog/system-tests-reviewers
manifests/python_lambda.yml                                             @DataDog/system-tests-reviewers
manifests/ruby_lambda.yml                                               @DataDog/system-tests-reviewers
tests/appsec/test_sca_reachability.py                                   @DataDog/asm-libraries
utils/_context/_scenarios/__init__.py                                   @DataDog/system-tests-core
utils/_context/_scenarios/aws_lambda.py                                 @DataDog/system-tests-core
utils/build/docker/java_lambda/java-alb-multi.Dockerfile                @DataDog/system-tests-reviewers
utils/build/docker/java_lambda/java-alb.Dockerfile                      @DataDog/system-tests-reviewers
utils/build/docker/java_lambda/java-apigw-http.Dockerfile               @DataDog/system-tests-reviewers
utils/build/docker/java_lambda/java-apigw-rest.Dockerfile               @DataDog/system-tests-reviewers
utils/build/docker/java_lambda/java-function-url.Dockerfile             @DataDog/system-tests-reviewers
utils/build/docker/java_lambda/pom.xml                                  @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/main.py                                 @DataDog/system-tests-reviewers
utils/scripts/libraries_and_scenarios_rules.yml                         @DataDog/system-tests-core

@datadog-prod-us1-3

datadog-prod-us1-3 Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Pipelines  Tests

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

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

Base automatically changed from clara.poncet/java-lambda-response-schema-weblog to main October 27, 2026 10:26
@claponcet
claponcet marked this pull request as ready for review October 28, 2026 23:46
@claponcet
claponcet requested review from a team as code owners October 28, 2026 23:46
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 28, 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-28T23:49:06.369293Z 2356c4d 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.

Comment thread utils/scripts/libraries_and_scenarios_rules.yml

@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.

Thanks for addressing the feedback, LGTM.

@claponcet
claponcet enabled auto-merge (squash) October 29, 2026 12:19
@claponcet
claponcet merged commit d0e9b0b into main Oct 29, 2026
819 checks passed
@claponcet
claponcet deleted the clara.poncet/java-lambda-sca branch October 29, 2026 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated The pull request includes a significant amount of AI-generated code java Pull requests that update Java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants