Probar la validación de nuestros endpoints

Clase 36 de 48 • Curso de Backend con Node.js 2019

Dante Calderón

student•

Para los que les sale el error joi.validate is not a function yo lo solucione cambiando la funcion validate a:

....
const validate = (data, schema) => {
  const { error } = joi.object(schema).validate(data)

  return error
}
....

o tambien pueden bajarle la version de @hapi/joi a la 15.0.3 como se indico en un comentario abajo

Arlex Felipe Llanos Betancourt

student•

Gran aporte, me sirvió mucho, gracias :D

Luis Miguel González

student•

Excelente aporte, muchas gracias!

Omar Flores Grimontt

student•

La versión 16.1.1 de Joi salió ayer con algunos cambios que rompen el código, por ende es muy probable que las validaciones de datos no te funcionen. Puedes encontrar acá más información al respecto.

Para corregir estos errores sólo debes meter tus objetos de schema que le pasas a la función validate en una función joi.object(). Y llamar el método validate desde el schema, ahora sólo va a recibir la data. Esto va a afectar principalmente tres porciones de código del proyecto, la primera son los schemas que declaras, quedarían así:

const joi = require('@hapi/joi')
const createMovieSchema = joi.object({
    title: movieTitleSchema.required(),
    year: movieYearSchema.required(),
    cover: movieCoverSchema.required(),
    description: movieDescriptionSchema.required(),
    duration: movieDurationSchema.required(),
    contentRating: movieContentRatingSchema.required(),
    source: movieSourceSchema.required(),
    tags: movieTagsSchema
})

La segunda son los métodos de las rutas de movies, quedarían así:

const express = require('express')
const joi = require('@hapi/joi')
const { MovieService } = require('../services')
const {
    movieIdSchema,
    createMovieSchema,
    updateMovieSchema
} = require('../utils/schemas/movies')

const validationHandler = require('../utils/middlewares/validation-handler')

function moviesApi(app) {
    const router = express.Router()
    const movieService = new MovieService()

    app.use('/api/movies', router)

    router.get(
        '/:movieId',
        validationHandler(joi.object({ movieId: movieIdSchema }), 'params'),
        async function(req, res, next) {
            const { movieId } = req.params
            try {
                const movie = await movieService.getMovie({ movieId })
                res.status(200).json({
                    data: movie,
                    message: 'movie by id'
                })
            } catch (error) {
                next(error)
            }
        }
    )
}

module.exports = moviesApi

Y la tercera es el método validate del validation handler que quedaría así:

function validate (data, schema) {
    const { error } = schema.validate(data)
    return error
}

Habiendo hecho estos cambios vas a poder validar tus datos.

O podrías instalar la versión 15.1.0 de @hapi/joi que es la que usa el profesor :P

OQO Digital

student•

Me funcionó functionvalidate(data, schema) { const { error } = schema.validate(data) returnerror } pero al querer usar el DELETE me arroja error. y ahí uso el que publicó otro:

function validate(data, schema) { const { error } = joi.object(schema).validate(data); return error; }

OQO Digital

student•

Ahhh, ya encontré mi error, no actualicé el delete con joi.object

jajajajaja

ahora con la solución del otro, no era necesario agregar el joi.object en cada uno, sino solo una vez. Ambas cosas sirven.

Manuel Ojeda

student•

Reto cumplido:

Agregue CORS al código usando:

npm i cors

Y siguiendo la documentación apliqué cors:

const express = require('express');
const cors = require('cors');
const app = express();

const { config } = require("./config/index");
const moviesApi = require('./routes/movies');

const { logErrors, wrapErrors, errorHandler } = require('./utils/middleware/errorHandlers');
const notFoundHandler = require('./utils/middleware/notFoundHandler');

// body.parser
app.use(express.json());
app.use(cors());

¿Porqué decidí utilizarlo? En clases anteriores quise probar la API desde otro proyecto en Vue solamente para poder consumir la API, obtuve un error de CORS, al aplicar este middleware ya pude consumir el proyecto API en mi proyecto Vue:

Vicente Daniel Zurita Salas

student•

excelente, no sabia que para eso era cors.

Juan Jose Vega

student•

crack

Milton Castañeda

student•

{ "id":"11fff70c-c312-4791-b8b4-1a0665edd79e", "title":"Siete minutos (Seven Minutes)", "year":2019, "cover":"http://dummyimage.com/800x600.png/5fa2dd/ffffff", "description":"Vestibulum ac est lacinia nisi venenatis tristique. Fusce congue, diam id ornare imperdiet, sapien urna pretium nisl, ut volutpat sapien arcu sed augue. Aliquam erat volutpat.\n\nIn congue. Etiam justo. Etiam pretium iaculis justo.", "duration":62, "contentRating":"PG", "source":"http://columbia.edu/eget/orci/vehicula/condimentum/curabitur/in/libero.html", "tags":["Action|Adventure|Comedy|Drama|War","Animation","Drama"] }

Irungaray Agustin

student•

Estaba esperando este comentario jajaj grgacias Milton!

platzerito02112019 platzerito02112019

student•

Al enviar el SEND en código como dice la clase me trono " TypeError: joi.validate is not a function " y lo solucione con esto: ~ npm uninstall --save @hapi/joi ~ npm install --save @hapi/joi@15.0.3

Luis Lazcano

student•

La librería de Joi ya actualizo por lo cuál se requieren algunos cambios en la sintaxis.

La sitnaxis de los schemas es diferente utils/schemas/movies.js

const createMovieSchema = joi.object({
  title: movieTitleSchema.required(),
  year: movieYearSchema.required(),
  cover: movieCoverSchema.required(),
  description: movieDescriptionSchema.required(),
  duration: movieDurationSchema.required(),
  contentRating: movieContentRatingSchema.required(),
  source: movieSourceSchema.required(),
  tags: movieTagSchema,
})

La forma de hacer la validación igual es de otra forma. utils/middleware/validationHandler.js

function validate(data, schema) {
  const { error } = schema.validate(data)
  return error;
}

Jonathan Jimenez Seguro

student•

gracias!!!

Alexander Nova Arevalo

student•

Thanks :)

Ernesto Palafox

student•

Si alguien más se pregunta por qué su error stack no se ve igual que el de @glrodasz

es por que por defecto el stack está desactivado ahora. Ahora hay que hacer:

  const { error } = schema.validate(data, { errors: { stack: true } })

David Behar

student•

Enqué parte hay que hacer eso, en el index?

platzerito02112019 platzerito02112019

student•

yo tambien pregunto lo mismo, donde va el codigo?

Edgar López Rangel

student•

{
  "title": "Notti bianche, Le (White Nights)",
  "year": 2019,
  "cover": "http://dummyimage.com/800x600.png/ff4444/ffffff",
  "description":
    "In hac habitasse platea dictumst. Etiam faucibus cursus urna. Ut tellus.",
  "duration": 66,
  "contentRating": "G",
  "source": "https://ovh.net/semper/rutrum/nulla/nunc.jsp",
  "tags": [
    "Action|Adventure",
    "Action|Adventure|Thriller",
    "Horror|Western",
    "Horror|Thriller",
    "Comedy|Romance|Sci-Fi",
    "Adventure|Animation|Children|Comedy|Fantasy",
    "Drama"
  ]
}

Geovanny Álvarez

student•

Fecha 28/julio/2020

Para los que tienen problemas y no les valida los datos JOI cambio algunas cosas y la forma en la que validamos los datos la cambie también porque no me validaba nada aquí dejo del código espero le sirva a alguno

Mi capa de schema se le agrega el joi.object({ Aquí adentro va el schema }) No se lo agregue a movieIdSchema , ya que se la capa en la parte de rutas

const joi = require('@hapi/joi');

const movieIdSchema = joi.string().regex(/^[0-9a-fA-F]{24}$/)
const movieTitleSchema = joi.string().max(80)
const movieYearSchema = joi.number().min(1888).max(2077)
const movieSchemaCover = joi.string().uri()
const movieDescriotionSchema = joi.string().max(300)
const movieDurationSchema = joi.string().min(1).max(300)
const movieSourceSchema = joi.string().uri()


const createMovieSchema = joi.object({
    title: movieTitleSchema.required(),
    year: movieYearSchema.required(),
    cover: movieSchemaCover.required(),
    description: movieDescriotionSchema.required(),
    duration: movieDurationSchema.required(),
    source: movieSourceSchema.required()
})


const updateMovieSchema = joi.object({
    title: movieTitleSchema,
    year: movieYearSchema,
    cover: movieSchemaCover,
    description: movieDescriotionSchema,
    duration: movieDurationSchema,
    source: movieSourceSchema
})

module.exports = {
    movieIdSchema,
    createMovieSchema,
    updateMovieSchema,
};

Mi capa de rutas Aquí agregue el joi.object({}) para movieIdSchema eso para cada vez que se quiera validar el ID el resto sigue igual dejo un ejemplo de como queda el resto abajo y recuerden requerir joi para poder usar el metodo .object()

router.get('/:movieId',  validationHandler(joi.object({ movieId: movieIdSchema }), 'params'), async (req, res, next) => {...

//Aqui pueden ver que se mantiene normal el resto igualmente recuerden requerir validationHandler
router.post('/', validationHandler(createMovieSchema), async (req, res, next) => {...

Mi capa de validación o validationHandler ahora retorno de modo asyncrono cambie check por data el resto es entendible si hay alguna otra duda comentarlo, yo lo hice de esta forma, ya que se me hizo más comprensible

const boom = require('@hapi/boom')

function validationHandler(schema, data = 'body'){
  return async (req,res,next) =>{
    try {
      await schema.validateAsync(req[data]);
      next();
    } catch (err) {
      next(boom.badRequest(err))
    }
  }
}


module.exports = {
  validationHandler
};

Jose Daniel Hernandez Quiceno

student•

Me sirvio! Excelente!

Fernando Cordero

student•

pero de donde sale validateAsync??

Laura Camila Pregonero

student•

Esta es mi corrección del codigo ya que no era un script() sino un number().

Milton Castañeda

student•

Pasar objeto js a json https://www.convertonline.io/convert/js-to-json

Marco Antonio Benites Espinoza

company_admin•

Reto:

const express = require('express');
const app = express();

const bodyParser = require('body-parser');

const { config } = require('./config/index');
const moviesApi = require('./routes/movies');
const notFoundHandler = require('./utils/middleware/notFoundHandler');

const {
    logErrors,
    wrapErrors,
    errorHandlers
} = require('./utils/middleware/errorHandlers');

**// parse application/x-www-form-urlencoded
app.use(bodyParser.urlencoded({ extended: false }));
// parse application/json
app.use(bodyParser.json());**

// Se agrega middleware de body-parser
//app.use(express.json());

// routes
moviesApi(app);

// Catch 404
app.use(notFoundHandler);

app.use(logErrors);
app.use(wrapErrors);
app.use(errorHandlers);

app.listen(config.port, function() {
    console.log(`Listening in http://localhost:${config.port}`);
});

Francisco Javier Suarez Verdugo

student•

Para las personas que se les este presentando el problema en postman: "message": "\"title\" is not allowed" Remuevan las llaves de los validationHandlers en los metodos .post y .put en el archivo routes/movies.js. Antes del cambio:

router.post('/', validationHandlers({createMovieSchema}), async function(req, res, next){

Despues del cambio:

router.post('/', validationHandlers(createMovieSchema), async function(req, res, next){

Espero que les sea de ayuda.

Pedro Roberto Capriles Morales

student•

Otra forma de resolver el problema joi.validate este caso para la versión joi ^17.1.1, gracias a los comentarios de abajo mas investigando un poco hice el siguiente refactor

schema/movies.js

const joi = require('@hapi/joi');

const movieIdSchema = joi.string().regex(/^[0-9a-fA-F]{24}$/);
const movieTitleSchema = joi.string().max(80);
const movieYearSchema = joi.number().min(1888).max(2077);
const movieCoverSchema = joi.string().uri();
const movieDescriptionSchema = joi.string().max(300);
const movieDurationSchema = joi.string().min(1).max(300);
const movieContentRatingSchema = joi.string().max(5);
const movieSourceSchema = joi.string().uri();
const movieTagsSchema = joi.array().items(joi.string().max(50));

const createMovieSchema = joi.object({
    title: movieTitleSchema.required(),
    year: movieYearSchema.required(),
    cover: movieCoverSchema.required(),
    description: movieDescriptionSchema.required(),
    duration: movieDurationSchema.required(),
    contentRating: movieContentRatingSchema.required(),
    source: movieSourceSchema.required(),
    tags: movieTagsSchema
});

const updateMovieSchema = joi.object({
    title: movieTitleSchema,
    year: movieYearSchema,
    cover: movieCoverSchema,
    description: movieDescriptionSchema,
    duration: movieDurationSchema,
    contentRating: movieContentRatingSchema,
    source: movieSourceSchema,
    tags: movieTagsSchema
});

module.exports = {
    movieIdSchema,
    createMovieSchema,
    updateMovieSchema
}

const boom = require('@hapi/boom');

const validate = (data, schema) => {
    const { error } = schema.validate(data);
    return error;
}

const validationHandler = (schema, check = 'body') => {
    return (request, response, next) => {
        const error = validate(request[check], schema);
        error ? next(boom.badRequest(error)) : next();
    };
}

module.exports = validationHandler;

Fuente: github

Carlos Enrique Ramírez Flores

student•

Tengo el siguiente error

Error: Object schema must be a valid object
    at new module.exports (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/hoek/lib/error.js:23:19)
    at module.exports (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/hoek/lib/assert.js:20:11)
    at internals.Base.method [as keys] (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/joi/lib/types/keys.js:241:17)
    at Object.args (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/joi/lib/types/keys.js:48:23)
    at Object.internals.generate (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/joi/lib/index.js:259:31)
    at Object.root.&lt;computed&gt; [as object] (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/joi/lib/index.js:58:30)
    at validate (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/utils/middleware/validationHandler.js:12:27)
    at /Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/utils/middleware/validationHandler.js:18:23
    at Layer.handle [as handle_request] (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/express/lib/router/layer.js:95:5)
    at next (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/express/lib/router/route.js:137:13)
Error: Object schema must be a valid object
    at new module.exports (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/hoek/lib/error.js:23:19)
    at module.exports (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/hoek/lib/assert.js:20:11)
    at internals.Base.method [as keys] (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/joi/lib/types/keys.js:241:17)
    at Object.args (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/joi/lib/types/keys.js:48:23)
    at Object.internals.generate (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/joi/lib/index.js:259:31)
    at Object.root.&lt;computed&gt; [as object] (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/@hapi/joi/lib/index.js:58:30)
    at validate (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/utils/middleware/validationHandler.js:12:27)
    at /Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/utils/middleware/validationHandler.js:18:23
    at Layer.handle [as handle_request] (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/express/lib/router/layer.js:95:5)
    at next (/Applications/XAMPP/xamppfiles/htdocs/platzi master/backend-con-nodejs/movies-api/node_modules/express/lib/router/route.js:137:13)

Mi Código

const boom = require('@hapi/boom');
const joi = require('@hapi/joi');

function validate(data, schema) {
    //console.log(schema);
    const { error } = joi.object(schema).validate(data);
    return error;
}

function validationHandler(schema, check = 'body') {
    return function (req, res, next) {
        const error = validate(req[check], schema);
        error ? next(boom.badRequest(error)) : next();
    }
}

module.exports = validationHandler;

Cesar Gonzalo Zorondo Garrido

student•

Puedes mostrar tu código del archivo schemas/movies.js, gracias.

Jorge Alonso Molina Chirinos

student•

¿A alguien más le aparece este error?

Fernando Cordero

student•

Chicos para el error de validación con Joi, lo único que tiene que modificar es : Con la versión 17.1.1:

function validate(data, schema) {
  const { error } = joi
    .object(schema)
    .validate(data, { errors: { stack: true } });
  return error;
}

Cualquier duda me escriben

Fernando Cordero

student•

Mi pregunta es: Como lleno mi base de datos con el moks? haciendo un post de cada movie?

Jose Daniel Molina

student•

Sí, debes enviar película por película, ya que no veo la forma de enviar lotes de información, creo que debe existir la manera pero por ahora no la encontré... O como tú eres el admin de la DB, puedes a manera de práctica llenar directamente la DB desde mongo Atlas.

Fernando Cordero

student•

@josedanielmolina hay una metodo de mongo insertMany, con eso podriamos poblar la db

Raymond Alexander Mansell Muñoz

student•

¿Qué pasa con los endpoints que requieren un ID válido con el formato que le dimos a movieIdSchema con el regex, pero enviamos un ID que no existe? Por ejemplo si eliminamos una pelicula, todavía podemos hacer peticiones a api/movies/:movieId con ese ID aunque ya no exista. Incluso en este punto podemos agregar películas por medio de peticiones PUT api/movies/:movieId si usamos ID's inexistentes. Y quedarían guardadas con campos vacíos, sin respetar los esquemas, ya que en updateMovieSchema no son requeridos.

Creo que sería bueno corregir services/movies.js y revisar si la pelicula existe en primer lugar, y de lo contrario enviar un 404. No sé si se haga más adelante en el curso.

Mauricio Rojas Nova

student•

Para los que tengan el error de "joi.validate is not a function", deben cambiar lo siguiente en el archivo validationHandlers.js:

function validate(data, schema) {
  const { error } = joi.object(schema).validate(data)
  return error;
}