Configurando Boom y nuestra página 404

Clase 25 de 43 • Curso de Express.js 2018

Jesus Olivares

student•

Me hubiera Gustado que los comentarios que agrega en el código de los videos tambien estuvieran en el código que bajamos en la seccion de enlaces.

Diego Hernán Moreno Amaya

student•

Por alguna extraña razón me estaba arrojando el error:

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

Y de ahí se desplegaban más errores parecidos. Para solucionar esto basta con poner un else de esta manera en el index.js:

app.use(function(req, res, next) {
  if (isRequestAjaxOrApi(req)) {
    const {
      output: { statusCode, payload }
    } = boom.notFound();
    res.status(statusCode).json(payload);
  } else {
    res.status(404).render("404");
  }
});

Diego Hernán Moreno Amaya

student•

Si están teniendo problemas porque la librería de boom está desactualizada utilicen @hapi/boom

Alejandro Valenzuela

student•

Buenas este video se me complico un poco..con conceptos que que no conozco, por ejemplo: 1-mutar 2- return {..err,stack} 3-rapeado(no estoy seguro como se escribe): si esta instanciado boom??? 4- const { output: {statusCode, payload} } = err; que hace? desde ya muchas gracias!pido disculpa si son conceptos básico que estoy desconociendo. Saludos.

Marcelo Gustavo Baez

student•

Primero de todo te recomiendo que si te cuestan algunos conceptos del lenguaje, tomes el curso de Fundamentos de Javascript (por cierto esta fenomenal!). Asi y todo voy a intentar despejarte algunas dudas: 1 y 2 - Mutar significa cambiar/alterar. La inmutabilidad es uno de los pilares de la programación Funcional, por ello el profesor recomienda utilizar el spread operator de Javascript para generar un nuevo objeto y a la vez indica la equivalencia de utilizar Object.assign. 3 - Wrappear es "envolver". 4 - Object destructuring es una caracteristica añadida a ECMAScript 2015. El profe aprovecha esto para generar un objeto a partir de las propiedades de otro existente.

Espero haber podido orientarte un poco mas

Carlos Enrique Ramírez Flores

student•

Hay un desorden con las actualizaciones de paquetes que se están usando.

Rubén Padilla

student•

Este curso es similar al Curso de Backend con Node.

Se trabajan con las mismas dependencias y la estructura es muy similar, te puedes apoyar para resolver dudas.

David Lara

student•

Marcador!, estudiar de nuevo esta clase

Santiago Ortiz Ceballos

student•

Alguien mas tiene inconvenientes con la siguiente parte?

/* Redirection routes */
app.use('/', function(req, res) {
    res.redirect('/products')
})

/* Not found middleware */
app.use(function(req, res, next) {
    if (isRequestAjaxOrApi(req)) {
        const {
            output: { statusCode, payload }
        } = boom.notFound()
        res.status(statusCode).json(payload)
    }
    res.status(404).render("404")
})

Todo me lo redirige a /products y jamas sale el 404, pero si pongo luego el redirect, ya se muestra el 404 y nunca redirige de '/' a '/products'

Gael Geovani Manzanares Quiñones

student•

Porque el middleware "wrapErrors" debe de ir en segundo lugar? Que importancia tiene el orden?

diego andres guzman

student•

hola, el manejador del error debe ser el ultimo inplementado porque captura el error sobre lo que susada por encima de el, si pones una funcion despues de la implentacion del error ya no va a capturar los errores de esa funcionalidad

jorge llanque

student•

Imagina como si fuera un try-catch

Naldo Duran

student•

✌

JAIME URQUIZU

student•

Lo peor no son los miles de errores que salen y que ni se comentan... El problema es que te descargas la carpeta 'master' para ver dónde puedes tener el error y... Ni siquiera funciona!!!!!

Esto es lo que aparece cuando intentas levantar la app que viene de la carpeta del profe...

[nodemon] 1.18.6
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: *.*
[nodemon] starting `node index index.js`    
events.js:183
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE :::8000
    at Object._errnoException (util.js:992:11)
    at _exceptionWithHostPort (util.js:1014:20)
    at Server.setupListenHandle [as _listen2] (net.js:1355:14)
    at listenInCluster (net.js:1396:12)
    at Server.listen (net.js:1480:7)
    at Function.listen (C:\platzi-express-configurando-boom-y-nuestra-pagina-404\ecommerce\node_modules\express\lib\application.js:618:24)
    at Object.<anonymous> (C:\platzi-express-configurando-boom-y-nuestra-pagina-404\ecommerce\index.js:58:20)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
[nodemon] app crashed - waiting for file changes before starting...

Nicolás Scandizzo

student•

a mi me funcion ok, cuando bajes el proyecto o lo clones desde github, verifica que tenes las dependencias instaladas, esto lo haces con :

npm install

seguido de

npm outdated

para verificar que dependencias están desactualizadas

y por ultimo actualizas con:

npm update

espero te sirva, saludos.

Ramdhei López Arcila

student•

Parece que tienes las dependencias desactualizadas, y hay un par de cosas que han cambiado con las actualizaciones.

Nicolas Cubillos Pardo

student•

La verdad esta entre los mejores cursos de platzi que he visto. Muy al detalle pero lo justo para entender porque funcionan las cosas y no le da miedo meterse a crear muchas cosas el. Para explicar perfecto pero en definitiva no para generar codigo para una aplicación propia. La mayoría de estos middlewares los he solventado con mongoose. Aqui el link si a alguien le interesa. https://mongoosejs.com/

Victor Hugo Cortés Posada

student•

Dice que el badRequest es el error adecuado pero creo que se deberia utilizar 422 badData.

400 Bad Request: El servidor no procesará la solicitud, porque no puede, o no debe, debido a algo que es percibido como un error del cliente (ej: solicitud malformada, sintaxis errónea, etc). La solicitud contiene sintaxis errónea y no debería repetirse.

422 Unprocessable Entity: La solicitud está bien formada pero fue imposible seguirla debido a errores semánticos.

que opinan los demas?

David Vargas Domínguez

student•

Cuando hago un request a la api para que me salga 404 sucede que estos dos me salen falsos !req.accepts(“html”) || req.xhr; y hace render a la página 404 en vez de darme el json

Será una configuración de postman?

app.use(function(req, res, next) {
  if (isRequestAjaxOrApi(req)) {
    const {
      output: { statusCode, payload }
    } = boom.notFound();

    res.status(statusCode).json(payload);
  }

  res.status(404).render("404");
});

function isRequestAjaxOrApi(req) {
  console.log(req.xhr);

  return !req.accepts("html") || req.xhr;
}

module.exports = isRequestAjaxOrApi;

Guillermo Rodas

teacher•

Para que req.xhr se marque como verdadero, necesitas agregar el header: X-Requested-With: XMLHttpRequest en tu peticion de postman.

Para que !req.accepts(“html”) se marque como verdadero, necesitas agregar en el header: Content-Type: application/json.

Cuando usas librerias de AJAX, como jQuery.ajax, axios, etc. generalmente suelen agregar el segundo header por ti, ya que por defecto esperan una respuesta de tipo JSON.

Johnner Alexander Gambacica Ortiz

student•

Buena clase =)

// Routes =>
// ....
// 404 =>
app.use((req, res, next) => {
  res.status(404).send('Super API not found');
});

Emmanuel Rodríguez

student•

En mi caso, no me agrada boom por el estar cargando con una implementación ajena a mi flujo de trabajo, por ello, yo implemento una interfaz de Error y utilizo el patrón de diseño factory:

/**
 * error component object for handle server errors
 * @typedef {Object} ErrorObject
 * @property {string} type - Indicates secope error by default ServerError
 * @property {number} code - Indicates code http error
 * @property {string} message - Indicates the effect error
 * @property {string|undefined} error - Indicates the main reason error
 */

export default class ServerError extends Error {
    /**
     * constructor server-error
     * @param {string} message - error effect message
     * @param {number} [code=500] - error code server error
     */
    constructor(message, code = 500, name = 'ServerError') {
        super(message);
        this.name = name;
        this.code = code;
    }

    /**
     * error object response
     * @param {string} error - error reason message
     * @returns {ErrorObject}
     */
    response(error) {
        return {
            type: this.name,
            code: this.code,
            message: this.message,
            error: error ? error : undefined
        };
    }
}

Daniel Esteban Santos Mendez

student•

¿Qué es request streaming?