"Unhandled Error" message while setting up server through http module in node js

Question

I have been writing the following code to set up a basic server through node.js. My code -

    const http  = require('http')
    const server = http.createServer((req, res) => {
    // console.log(req)
      if(req.url === '/'){
          res.end("Welcome to our home page")
      }
      if(req.url === '/aboutus'){
          res.end("Welcome to about us page")
      }
      res.end(`
          <h1> OOPS !!! </h1>
          <p>You have requested for any wrong address </p>
          <a href = "/">Get back to HOME Page</a>
     `)
    })
    server.listen(5000)

But this code while being run is giving the following error in my console. Error --

node:events:368
      throw er; // Unhandled 'error' event
      ^

Error [ERR_STREAM_WRITE_AFTER_END]: write after end
    at new NodeError (node:internal/errors:371:5)
    at ServerResponse.end (node:_http_outgoing:846:15)
    at Server.<anonymous> (/home/tirtharaj/MERN/Node/tutorial(fcc)/8-Built-in Modules/6-httpModule(brief).js:10:9)
    at Server.emit (node:events:390:28)
    at parserOnIncoming (node:_http_server:951:12)
    at HTTPParser.parserOnHeadersComplete (node:_http_common:128:17)
Emitted 'error' event on ServerResponse instance at:
    at emitErrorNt (node:_http_outgoing:726:9)
    at processTicksAndRejections (node:internal/process/task_queues:84:21) {
  code: 'ERR_STREAM_WRITE_AFTER_END'
}

I am not able to find any error in my code. Any refresh in the web browser in giving error. So please help me run this code.

supertux · Accepted Answer · 2022-01-22 15:46:39Z

2

This will solve your problem, you need to have all the code wrapped in if else statements so that it can always terminate successfully with only one possible path through the code.

const http = require("http");
const server = http.createServer((req, res) => {
  if (req.url === "/") {
    res.end("Welcome to our home page");
  } else if (req.url === "/aboutus") {
    res.end("Welcome to about us page");
  } else {
    res.end(`
        <h1> OOPS !!! </h1>
        <p>You have requested for any wrong address </p>
        <a href = "/">Get back to HOME Page</a>
    `);
  }
});
server.listen(5000);

Personally, I would use a switch case statement for this to keep it nice and clean.

const http = require("http");
const server = http.createServer((req, res) => {
  switch (req.url) {
    case "/":
      res.end("Welcome to our home page");
      break;

    case "/aboutus":
      res.end("Welcome to about us page");
      break;

    default:
      res.end(`
        <h1> OOPS !!! </h1>
        <p>You have requested for any wrong address </p>
        <a href = "/">Get back to HOME Page</a>
      `);
      break;
  }
});
server.listen(5000);

edited Jan 22, 2022 at 15:46

answered Jan 22, 2022 at 15:40

supertux

2,2691 gold badge17 silver badges23 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Tirtharaj Sen Over a year ago

Thank You very much !!! This method actally works . I didn't actually knew that we always need to wrap the code in if else statement to terminate if. Thanks a lot.

Collectives™ on Stack Overflow

"Unhandled Error" message while setting up server through http module in node js

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related