0

estoy trabajando con AJV Schema para corroborar que el tipo de dato recibido es correcto. Si bien la clave la verifica bien, la función que analiza el formato del email es ignorado. Alguna sugerencia?

const Ajv = require("ajv").default
const AjvFormats = require('ajv-formats');
const ajv = new Ajv()
AjvFormats(ajv);

const email = { email: '' }

const schema = {
    type: "object",
    properties: {
        email: {type: "string"}
    },
    required: ["email"]
}

function check(data){

    const emailRegex = new RegExp('/^(([^<>()[\]\\.,;:\s@"]+(\.[^<>()[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/')
    ajv.addFormat('email-address', {
        validate: (format) => emailRegex.test(format)
    })

    const valid = ajv.validate(schema, data)
    !valid ? console.log(ajv.errors) : console.log(data)

}

check(email)
2
  • 1
    dentro de check(), email y data son { email: '' }, y además data no lo usas.
    – Emeeus
    Commented el 22 jun. 2021 a las 21:42
  • Hice los cambios que ayer pasé por alto cuando subi esta publicación, pero no funciona. Justamente dejé a propósito el campo de email vacio para que salte a la primera un mensaje de error. En cambio, si la key email no está o está mal escrita funciona.
    – Martín LC
    Commented el 23 jun. 2021 a las 14:19

1 respuesta 1

2

El problema reside en que en el schema no estás definiendo explícitamente que la propiedad email tendrá un formato, solo que será de tipo string. Por lo que el schema debería ser definido de la siguiente manera:

const schema = {
    type: 'object',
    properties: {
        email: {type: 'string', format: 'email-format'}
    },
    required: ['email']
}

Luego entonces al hacer tu validación, deberías ser capaz de obtener el resultado que esperas.

[{
  instancePath: "/email",
  schemaPath: "#/properties/email/format",
  keyword: "format",
  params: { format: "email-format" },
  message: "must match format \"email-format\""
}]

Runkit con el ejemplo funcionando como se espera y unas ligeras modificaciones.

Edit: Desconozco el funcionamiento a fondo de addFormat, pero revisando el ejemplo de la documentación noté que al usar una regex para un formato, la aplican directamente sin construirla con RegExp(). Entonces la parte de añadir el formato quedaría así:

const emailRegex = /^(([^<>()[\]\\.,;:\s@"]+(\.[^<>()[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;

ajv.addFormat('email-format', emailRegex);

Con el cambio anterior obtienes la siguiente salida:

// { email: 'email' }:
// [{
//   instancePath: "/email",
//   schemaPath: "#/properties/email/format",
//   keyword: "format",
//   params: { format: "email-format" },
//   message: "must match format \"email-format\""
// }]

// { email: '[email protected]' }:
// { email: "[email protected]" }

// { email: 'e [email protected]' }:
// [{
//   instancePath: "/email",
//   schemaPath: "#/properties/email/format",
//   keyword: "format",
//   params: { format: "email-format" },
//   message: "must match format \"email-format\""
// }]

Para estos casos básicos funciona, tienes que tener en cuenta que intentar validar un email usando solo una regex puede ser algo complejo, así que eso ya depende de la regex que utilices.

5
  • 1
    Hola Ricardo! Pensé que lo había solucionado pero ahora todo el tiempo me arroja el resultado de que el formato no es válido!
    – Martín LC
    Commented el 24 jun. 2021 a las 19:22
  • Puedes revisar mi actualización. También puedes checar el runkit actualizado. Commented el 24 jun. 2021 a las 21:34
  • Ahora si! Muy bueno Ricardo, muchísimas gracias por el aporte y la resolución. De paso también terminé mejorando la expresión regular: /\S+@\S+\.\S+/
    – Martín LC
    Commented el 25 jun. 2021 a las 16:10
  • Genial, un gusto poder haber ayudado. Solo como comentario, ojo con el cambio de tu regex, el correo e [email protected] debería ser marcado como inválido (debido al espacio). Commented el 25 jun. 2021 a las 16:57
  • No me había dado cuenta de ese detalle. Fenómeno!!
    – Martín LC
    Commented el 25 jun. 2021 a las 20:51

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.