Skip to content

Commit

Permalink
doc: update code examples in domain.md
Browse files Browse the repository at this point in the history
* var -> const
* string concatenation -> template strings

PR-URL: #11110
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
vsemozhetbyt authored and MylesBorins committed Mar 9, 2017
1 parent d729e52 commit 61ac334
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions doc/api/domain.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,13 +47,13 @@ For example, this is not a good idea:
```js
// XXX WARNING! BAD IDEA!

var d = require('domain').create();
const d = require('domain').create();
d.on('error', (er) => {
// The error won't crash the process, but what it does is worse!
// Though we've prevented abrupt process restarting, we are leaking
// resources like crazy if this ever happens.
// This is no better than process.on('uncaughtException')!
console.log('error, but oh well', er.message);
console.log(`error, but oh well ${er.message}`);
});
d.run(() => {
require('http').createServer((req, res) => {
Expand Down Expand Up @@ -104,9 +104,9 @@ if (cluster.isMaster) {
// worker processes to serve requests. How it works, caveats, etc.

const server = require('http').createServer((req, res) => {
var d = domain.create();
const d = domain.create();
d.on('error', (er) => {
console.error('error', er.stack);
console.error(`error ${er.stack}`);

// Note: we're in dangerous territory!
// By definition, something unexpected occurred,
Expand All @@ -115,7 +115,7 @@ if (cluster.isMaster) {

try {
// make sure we close down within 30 seconds
var killtimer = setTimeout(() => {
const killtimer = setTimeout(() => {
process.exit(1);
}, 30000);
// But don't keep the process open just for that!
Expand All @@ -135,7 +135,7 @@ if (cluster.isMaster) {
res.end('Oops, there was a problem!\n');
} catch (er2) {
// oh well, not much we can do at this point.
console.error('Error sending 500!', er2.stack);
console.error(`Error sending 500! ${er2.stack}`);
}
});

Expand All @@ -156,7 +156,7 @@ if (cluster.isMaster) {
// This part isn't important. Just an example routing thing.
// You'd put your fancy application logic here.
function handleRequest(req, res) {
switch(req.url) {
switch (req.url) {
case '/error':
// We do some async stuff, and then...
setTimeout(() => {
Expand Down Expand Up @@ -239,16 +239,16 @@ serverDomain.run(() => {
// req and res are also created in the scope of serverDomain
// however, we'd prefer to have a separate domain for each request.
// create it first thing, and add req and res to it.
var reqd = domain.create();
const reqd = domain.create();
reqd.add(req);
reqd.add(res);
reqd.on('error', (er) => {
console.error('Error', er, req.url);
try {
res.writeHead(500);
res.end('Error occurred, sorry.');
} catch (er) {
console.error('Error sending 500', er, req.url);
} catch (er2) {
console.error('Error sending 500', er2, req.url);
}
});
}).listen(1337);
Expand Down

0 comments on commit 61ac334

Please sign in to comment.