What's The Best Way To Deal With An Error In The Server Side And In The Client Side Using Nodejs + Express

I'd like to know the best way to deal with errors in a response - request. I have this route that receive a request: app.get('/getInfo', function (req, res, next) { let obj = {

Solution 1:

For this case

res.send({error: "The data wasn't load"}) 

vs

res.status(500).send({error: "The data wasn't load"})

send a status is just more detailed, but both are ok. check Proper way to set response status and JSON content

For this case, depends on what you need

then(resp => {
            if(resp.status === 200){
                 this.appInfoHandler(resp.data)
            }else if (resp.status === 400){
                  //print error message on web interface
            }else if (resp.status === 500){
                  //print error message on web interface
          })
          .catch(function (error) {    
            console.log(error)
          })

vs

getInfo () {
    axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
        this.appInfoHandler(resp.data)
      })
      .catch(function (error) {    
        //print error message on web interface
      })
  }

You can handle all the errors sending them to the catch block

else if (resp.status === 400){
                  //print error message on web interface

not printing the error in here but throwing a new error that will be send it to the catch block

throw new ApiError("UserNotFount",400,"not found");

throw new Error('Error 400, not found');

Solution 2:

For this case

res.send({error: "The data wasn't load"}) 

vs

res.status(500).send({error: "The data wasn't load"})

I would suggest sending error as well as status code because that will be more descriptive for the client.

and for the second case

getInfo () {
      axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
            if(resp.status === 200){
                 this.appInfoHandler(resp.data)
            }else if (resp.status === 400){
                  //print error message on web interface
            }else if (resp.status === 500){
                  //print error message on web interface
          })
          .catch(function (error) {    
            console.log(error)
          })

vs

getInfo () {
    axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
        this.appInfoHandler(resp.data)
      })
      .catch(function (error) {    
        //print error message on web interface
      })
  }

In this case I would suggest to use the catch block directly whenever you get an error because response status depends on error but not the other way around

Solution 3:

As a beginner working on a REST Api, you should take a look at a guidelines - microsoft's are pretty legit: https://docs.microsoft.com/en-us/azure/architecture/best-practices/api-design.

Basically, you need to return the correct HTTP code for each request, take a look at https://http.cat/ - for example if the request is malformed, return 400, and if the user is unauthorized return 401:

if (!req.body.name) {
  res.status(400).send({ error: 'missing user name' }); // 400 bad request
}
const user = getUser(req.body.name, req.body.pass);

if(!user) {
  res.status(401).send({ error: 'user does not exist' }); // 401 unauthorized
}

try {
  const token = createToken(user);
  // better to set a cookie
  res.status(200).send({ token }); // 200 success
} catch(e) {
  res.status(500).send({ erroe: e.message }); // 500 internal error
}

if(isTeapot) {
  res.status(418).send({ error: 'I can only make tea' }); // 418 teapot, totally real
}

To make things easier there are a lot of libraries to help you generate better error messages and handle errors better, one of my favorites is celebrate

Solution 4:

Any status code other that 200 would mean unsuccessful so you dont need to use those if-else statements. The better alternative is to catch the error and send it with response as it is. The benefit is that you would receive the type of error occured without hardcoding the status codes. (for ex, we take the status code here to be 400 unsuccessful)

 .catch(function (error) {    
        //print error message on web interface
        res.status(400).send(JSON.stringify(error, undefined, 2));
      });

By using the stringify method you can print the exact error on the console also.

.catch(function (error) {    
            console.log(JSON.stringify(error, undefined, 2));
          });

The parameters in the stringify method here are:

1. error object

2. undefined: The array which contains the keys for filtering the keys in the object(here, error). All those keys present in this array are only the ones not filtered out.

3. 2: It is used to introduce whitespace in object representation