Skip to content

fix: allow the status code be explicity set.#36

Merged
wanlin31 merged 1 commit intogoogle:mainfrom
wanlin31:fix-typo
Aug 21, 2025
Merged

fix: allow the status code be explicity set.#36
wanlin31 merged 1 commit intogoogle:mainfrom
wanlin31:fix-typo

Conversation

@wanlin31
Copy link
Copy Markdown
Collaborator

@wanlin31 wanlin31 commented Aug 20, 2025

This cl is a follow up for #35

Change included:

  • allow the status code be set explicitly. net/http server will automatically send a 200 OK status code by default, that is the reason this was not caught by the test.
  • update the developer guide

Test: local build and print out the headers and status code.

@wanlin31 wanlin31 requested a review from hkt74 August 20, 2025 22:42
@wanlin31 wanlin31 marked this pull request as ready for review August 20, 2025 22:48
@wanlin31 wanlin31 merged commit d44a1d5 into google:main Aug 21, 2025
2 checks passed
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