Skip to content

[nodejs] return body with headers on / weblog endpoint#6306

Merged
CarlesDD merged 14 commits intomainfrom
romain.marcadier/APPSEC-61286/nodejs-weblog-headers
Feb 18, 2026
Merged

[nodejs] return body with headers on / weblog endpoint#6306
CarlesDD merged 14 commits intomainfrom
romain.marcadier/APPSEC-61286/nodejs-weblog-headers

Conversation

@RomainMuller
Copy link
Contributor

Summary

Update Node.js weblog variants to return "Hello world!\n" with Content-Type: text/plain from the / endpoint, matching the updated weblog specification. Express/Fastify auto-set Content-Length.

Variants fixed

express, express4-typescript, fastify, anthropic_app, openai_app, google_genai_app, nextjs

The AI app variants (anthropic_app, openai_app, google_genai_app) were missing the / endpoint entirely. nextjs is a special case — it serves HTML due to React page routing.

christophe-papazian and others added 10 commits February 13, 2026 11:16
JJ-Change-Id: wkrxvy
…th` on `/`

Update all Node.js weblog variants to return `"Hello world!\n"` with
`text/plain` content-type from the `/` endpoint, matching the updated
weblog specification. Express/Fastify auto-set `Content-Length`.

Variants fixed: `express`, `express4-typescript`, `fastify`,
`anthropic_app`, `openai_app`, `google_genai_app`, `nextjs`.

JJ-Change-Id: skymyr
@RomainMuller RomainMuller requested review from a team as code owners February 13, 2026 14:39
@RomainMuller RomainMuller changed the title fix(nodejs weblog): return body with headers on / endpoint [nodejs] return body with headers on / weblog endpoint Feb 13, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

CODEOWNERS have been resolved as:

utils/build/docker/nodejs/express/app.js                                @DataDog/dd-trace-js @DataDog/system-tests-core
utils/build/docker/nodejs/express4-typescript/app.ts                    @DataDog/dd-trace-js @DataDog/system-tests-core
utils/build/docker/nodejs/fastify/app.js                                @DataDog/dd-trace-js @DataDog/system-tests-core
utils/build/docker/nodejs/nextjs/src/app/page.js                        @DataDog/dd-trace-js @DataDog/system-tests-core

Base automatically changed from romain.marcadier/APPSEC-61286/go-weblog-headers to christophe-papazian/test_content_type February 13, 2026 15:44
@RomainMuller RomainMuller requested a review from a team as a code owner February 13, 2026 15:44
Base automatically changed from christophe-papazian/test_content_type to main February 13, 2026 19:33
@RomainMuller RomainMuller requested review from a team as code owners February 13, 2026 19:33
@RomainMuller RomainMuller removed the request for review from a team February 13, 2026 19:33
@RomainMuller RomainMuller requested review from Yun-Kim, avara1986, daniel-romano-DD, dmehala and manuel-alvarez-alvarez and removed request for a team February 13, 2026 19:33
@CarlesDD CarlesDD merged commit 5ca858f into main Feb 18, 2026
179 checks passed
@CarlesDD CarlesDD deleted the romain.marcadier/APPSEC-61286/nodejs-weblog-headers branch February 18, 2026 10:25
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.

6 participants

Comments