nginx: enable test for http.endpoint tag

[cataphract]

Motivation

Enables the http.endpoint tag tests. See DataDog/nginx-datadog#267 and DataDog/dd-trace-cpp#260

Changes

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

• If PR title starts with [<language>], double-check that only <language> is impacted by the change
• No system-tests internal is modified. Otherwise, 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 (or removed)?
  • Get a review from R&P team

[github-actions]

CODEOWNERS have been resolved as:

manifests/cpp_nginx.yml                                                 @DataDog/dd-trace-cpp
tests/test_resource_renaming.py                                         @DataDog/system-tests-core
utils/build/docker/cpp_nginx/nginx/nginx-waf.conf                       @DataDog/system-tests-core
utils/build/docker/cpp_nginx/nginx/nginx.conf                           @DataDog/system-tests-core

[datadog-prod-us1-3]

 

✨ Fix all issues with BitsAI

⚠️ Warnings

🚦 4 Pipeline jobs failed

Testing the test | System Tests (cpp_nginx, dev) / End-to-end #1 / nginx 1     

Testing the test | System Tests (cpp_nginx, dev) / End-to-end #2 / nginx 2     

Testing the test | System Tests (php, prod) / End-to-end #2 / apache-mod-7.3 2     

View all 4 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

Useful? React with 👍 / 👎

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: dd4b1ac | Docs | Datadog PR Page | Give us feedback!}

[cataphract]

RFC-1051 is ambiguous on whether truncated paths should have a trailing slash.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 832fb7cd05

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Add resource_renaming to the active nginx config

For the versions newly enabled in manifests/cpp_nginx.yml (v1.10.0+), utils/build/docker/cpp_nginx/install_ddtrace.sh symlinks /etc/nginx/nginx.conf to nginx.conf.waf whenever the module version is greater than v1.1.0; this added location is in the no-WAF config and is therefore not loaded. I checked nginx-waf.conf, and /resource_renaming/... still falls through to location / with try_files $uri =404, so the newly enabled cpp_nginx resource-renaming tests run without the intended endpoint handling and will keep failing/returning 404s. Please mirror this block in nginx-waf.conf or change the selected config.

Useful? React with 👍 / 👎.

[cataphract]

not strictly required for the tests to pass, since the RFC has no carve out for 404 responses, but advice followed

[datadog-system-tests-org]

⚠️ Warnings

🚦 4 Pipeline jobs failed

Testing the test | System Tests (cpp_nginx, dev) / End-to-end #1 / nginx 1     

Testing the test | System Tests (cpp_nginx, dev) / End-to-end #2 / nginx 2     

Testing the test | System Tests (php, prod) / End-to-end #2 / apache-mod-7.3 2     

View all 4 failed jobs.

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: dd4b1ac | Docs | Give us feedback!}