Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit fbb36ed

Browse filesBrowse files
jasnelladdaleax
authored andcommitted
doc: add comment to example about 2xx status codes
Fixes: #29714 PR-URL: #34223 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
1 parent f2f1537 commit fbb36ed
Copy full SHA for fbb36ed

File tree

Expand file treeCollapse file tree

1 file changed

+2
-0
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+2
-0
lines changed
Open diff view settings
Collapse file

‎doc/api/http.md‎

Copy file name to clipboardExpand all lines: doc/api/http.md
+2Lines changed: 2 additions & 0 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -2108,6 +2108,8 @@ http.get('http://nodejs.org/dist/index.json', (res) => {
21082108
const contentType = res.headers['content-type'];
21092109

21102110
let error;
2111+
// Any 2xx status code signals a successful response but
2112+
// here we're only checking for 200.
21112113
if (statusCode !== 200) {
21122114
error = new Error('Request Failed.\n' +
21132115
`Status Code: ${statusCode}`);

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.