Don't write a response body for null body status codes in sendStatus

[programmer4285]

resolves #335

[Copilot]

Pull request overview

This PR updates sendStatus to avoid emitting a reason-phrase response body for HTTP status codes that must not include content (1xx, 204, 205, 304), aligning the framework’s behavior with RFC 9110 semantics.

Changes:

• Added statusBodyLookup(status) utility to return an empty string for no-body status codes.
• Updated RESPONSE.sendStatus() to use statusBodyLookup instead of statusLookup.
• Added unit tests covering statusBodyLookup for common status codes.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
src/lib/utils.js	Adds statusBodyLookup to suppress bodies for no-body HTTP status codes.
src/lib/response.js	Switches sendStatus to use statusBodyLookup when generating the response body.
tests/utils.unit.js	Adds unit tests for the new statusBodyLookup helper.

[programmer4285]

hi @naorpeled, what is the process for getting this PR merged? should i address the comments from copilot?

[naorpeled]

hi @naorpeled, what is the process for getting this PR merged? should i address the comments from copilot?

Hey,
Yes, once the comments are addressed I'll merge the PR 🙏

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

[naorpeled]

Thanks @programmer4285 !