MONEI / monei-node-sdk

MONEI Node.js SDK
https://docs.monei.com
5 stars 0 forks source link

Hay un console log en monei.ts #2

Closed pabloaguilarmartinez closed 2 months ago

pabloaguilarmartinez commented 2 months ago

Hola, he visto en nuestra aplicación, que estamos usando monei, que se está logeando algo. Pensaba que era nuestro, pero parece que es la librería. No sé si puede ser un dato importante o algo que se os ha pasado en una prueba, por si es conveniente quitarlo. Está en el método verifiSignature

verifySignature(body: string, signature: string) {
    const parts = signature.split(',').reduce<Record<string, string>>((result, part) => {
      const [key, value] = part.split('=');
      result[key] = value;
      return result;
    }, {});
    const hmac = crypto
      .createHmac('SHA256', this.apiKey)
      .update(`${parts.t}.${body}`)
      .digest('hex');

    console.log(hmac, parts.v1);

    if (hmac !== parts.v1) {
      throw new Error('Signature verification failed.');
    }

    return JSON.parse(body);
  }
jimmyn commented 2 months ago

Hola Pablo, gracias por señalarlo. Debería estar corregido en la última versión.