Skip to content

Commit aabfdef

Browse files
deokjinkimtargos
authored andcommitted
doc: use console.error for error case in fs, https, net and process
console.error is more suitable than console.log for error case. PR-URL: #45606 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Kohei Ueno <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Paolo Insogna <[email protected]>
1 parent 845f805 commit aabfdef

File tree

4 files changed

+15
-15
lines changed

4 files changed

+15
-15
lines changed

doc/api/fs.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -941,15 +941,15 @@ try {
941941
await copyFile('source.txt', 'destination.txt');
942942
console.log('source.txt was copied to destination.txt');
943943
} catch {
944-
console.log('The file could not be copied');
944+
console.error('The file could not be copied');
945945
}
946946

947947
// By using COPYFILE_EXCL, the operation will fail if destination.txt exists.
948948
try {
949949
await copyFile('source.txt', 'destination.txt', constants.COPYFILE_EXCL);
950950
console.log('source.txt was copied to destination.txt');
951951
} catch {
952-
console.log('The file could not be copied');
952+
console.error('The file could not be copied');
953953
}
954954
```
955955

doc/api/https.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ let https;
2424
try {
2525
https = require('node:https');
2626
} catch (err) {
27-
console.log('https support is disabled!');
27+
console.error('https support is disabled!');
2828
}
2929
```
3030

@@ -42,7 +42,7 @@ let https;
4242
try {
4343
https = await import('node:https');
4444
} catch (err) {
45-
console.log('https support is disabled!');
45+
console.error('https support is disabled!');
4646
}
4747
```
4848

doc/api/net.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -409,7 +409,7 @@ after a certain amount of time:
409409
```js
410410
server.on('error', (e) => {
411411
if (e.code === 'EADDRINUSE') {
412-
console.log('Address in use, retrying...');
412+
console.error('Address in use, retrying...');
413413
setTimeout(() => {
414414
server.close();
415415
server.listen(PORT, HOST);

doc/api/process.md

+10-10
Original file line numberDiff line numberDiff line change
@@ -3186,7 +3186,7 @@ if (process.getegid && process.setegid) {
31863186
process.setegid(501);
31873187
console.log(`New gid: ${process.getegid()}`);
31883188
} catch (err) {
3189-
console.log(`Failed to set gid: ${err}`);
3189+
console.error(`Failed to set gid: ${err}`);
31903190
}
31913191
}
31923192
```
@@ -3200,7 +3200,7 @@ if (process.getegid && process.setegid) {
32003200
process.setegid(501);
32013201
console.log(`New gid: ${process.getegid()}`);
32023202
} catch (err) {
3203-
console.log(`Failed to set gid: ${err}`);
3203+
console.error(`Failed to set gid: ${err}`);
32043204
}
32053205
}
32063206
```
@@ -3231,7 +3231,7 @@ if (process.geteuid && process.seteuid) {
32313231
process.seteuid(501);
32323232
console.log(`New uid: ${process.geteuid()}`);
32333233
} catch (err) {
3234-
console.log(`Failed to set uid: ${err}`);
3234+
console.error(`Failed to set uid: ${err}`);
32353235
}
32363236
}
32373237
```
@@ -3245,7 +3245,7 @@ if (process.geteuid && process.seteuid) {
32453245
process.seteuid(501);
32463246
console.log(`New uid: ${process.geteuid()}`);
32473247
} catch (err) {
3248-
console.log(`Failed to set uid: ${err}`);
3248+
console.error(`Failed to set uid: ${err}`);
32493249
}
32503250
}
32513251
```
@@ -3276,7 +3276,7 @@ if (process.getgid && process.setgid) {
32763276
process.setgid(501);
32773277
console.log(`New gid: ${process.getgid()}`);
32783278
} catch (err) {
3279-
console.log(`Failed to set gid: ${err}`);
3279+
console.error(`Failed to set gid: ${err}`);
32803280
}
32813281
}
32823282
```
@@ -3290,7 +3290,7 @@ if (process.getgid && process.setgid) {
32903290
process.setgid(501);
32913291
console.log(`New gid: ${process.getgid()}`);
32923292
} catch (err) {
3293-
console.log(`Failed to set gid: ${err}`);
3293+
console.error(`Failed to set gid: ${err}`);
32943294
}
32953295
}
32963296
```
@@ -3321,7 +3321,7 @@ if (process.getgroups && process.setgroups) {
33213321
process.setgroups([501]);
33223322
console.log(process.getgroups()); // new groups
33233323
} catch (err) {
3324-
console.log(`Failed to set groups: ${err}`);
3324+
console.error(`Failed to set groups: ${err}`);
33253325
}
33263326
}
33273327
```
@@ -3334,7 +3334,7 @@ if (process.getgroups && process.setgroups) {
33343334
process.setgroups([501]);
33353335
console.log(process.getgroups()); // new groups
33363336
} catch (err) {
3337-
console.log(`Failed to set groups: ${err}`);
3337+
console.error(`Failed to set groups: ${err}`);
33383338
}
33393339
}
33403340
```
@@ -3365,7 +3365,7 @@ if (process.getuid && process.setuid) {
33653365
process.setuid(501);
33663366
console.log(`New uid: ${process.getuid()}`);
33673367
} catch (err) {
3368-
console.log(`Failed to set uid: ${err}`);
3368+
console.error(`Failed to set uid: ${err}`);
33693369
}
33703370
}
33713371
```
@@ -3379,7 +3379,7 @@ if (process.getuid && process.setuid) {
33793379
process.setuid(501);
33803380
console.log(`New uid: ${process.getuid()}`);
33813381
} catch (err) {
3382-
console.log(`Failed to set uid: ${err}`);
3382+
console.error(`Failed to set uid: ${err}`);
33833383
}
33843384
}
33853385
```

0 commit comments

Comments
 (0)