Skip to content

add node:http and node:https documentation #24268

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 18, 2025

Conversation

anonrig
Copy link
Member

@anonrig anonrig commented Aug 8, 2025

Summary

Adds the documentation for node http and https modules.

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.

Copy link
Contributor

github-actions bot commented Aug 8, 2025

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/workers/ @cloudflare/workers-docs, @GregBrimble, @irvinebroque, @mikenomitch, @korinne, @WalshyDev, @cloudflare/deploy-config, @cloudflare/pcx-technical-writing, @kodster28, @cloudflare/wrangler, @cloudflare/workers-runtime-1, @cloudflare/wrangler

@jasnell
Copy link
Contributor

jasnell commented Aug 8, 2025

We should probably mark this draft or hold off on merging until the server side stuff is actually rolled out without the experimental flag.

@anonrig anonrig force-pushed the yagiz/add-nodejs-http-https-modules branch from 68d2271 to 6810838 Compare August 8, 2025 19:54
@anonrig anonrig force-pushed the yagiz/add-nodejs-http-https-modules branch 3 times, most recently from 68b9d8f to aa9d87d Compare August 18, 2025 15:19
@anonrig anonrig requested a review from jasnell August 18, 2025 15:27
Copy link
Contributor

@irvinebroque irvinebroque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ so you have it, few comments, @mikenomitch

@anonrig anonrig force-pushed the yagiz/add-nodejs-http-https-modules branch 3 times, most recently from d876f5d to fb8f91e Compare August 18, 2025 19:41
@anonrig anonrig force-pushed the yagiz/add-nodejs-http-https-modules branch from fb8f91e to 7b3eb4e Compare August 18, 2025 22:31
@anonrig anonrig merged commit 0004093 into production Aug 18, 2025
8 checks passed
@anonrig anonrig deleted the yagiz/add-nodejs-http-https-modules branch August 18, 2025 23:02
@mikenomitch
Copy link
Contributor

@@ -129,7 +152,37 @@ export default {
// ...
}
}
```

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing ```

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product:workers Related to Workers product size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants