Discussion :

[tony060]

Bonjour,

J'ai une erreur sur ma route put lorsque j'essaye de modifié un utilisateur:

(node:19528) UnhandledPromiseRejectionWarning: Error: data and salt arguments required

j'utilise bcrypt. Je ne vois pas d'où peut bien venir l'erreur. J'ai essayé différentes choses que j'ai pu lire sur des forums mais ça ne fonctionne pas.
Qui pourrait me dire d'où vient le problème ?
Je vous remercie par avance.

Voici mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
exports.modifyUser =  async (req, res, next) => {
    const id = parseInt(req.params.id)
    const { username , email, lastname, firstname } = req.body 
    const password_regex = /^(?=.*[0-9]+.*)(?=.*[a-zA-Z]+.*)[0-9a-zA-Z]{8,}$/; 
    const encryptedPassword = await bcrypt.hash(req.body.password, 10)
    const password = req.body.password
    connection.query('UPDATE users SET username = ?, email = ?, lastname = ?, firstname = ?, password = ? WHERE id =? ',[username, email, lastname, firstname ,encryptedPassword, id], (error, results) => {
        if (error) {
            res.status(401).json({err: 'la modification à échouée !' })
        }else{ 
            res.status(200).json({"Utilisateur modifié par": `${id}`}) 
        }        
    })  
};

[djedjethai]

Bonjour,

Le message d erreur indique qu'une promise 'reject' et l'erreur n'est pas attrapee. Utilise avec async/await les methodes retournant des promises doivent etre placees dans des try{}catch(e){}.
Dans ce cas l'erreur provenant de bcrypt n'est pas attrapee. Ceci devrait resoudre le probleme.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
exports.modifyUser =  async (req, res, next) => {
    try{
    const id = parseInt(req.params.id)
    const { username , email, lastname, firstname } = req.body 
    const password_regex = /^(?=.*[0-9]+.*)(?=.*[a-zA-Z]+.*)[0-9a-zA-Z]{8,}$/; 
    const encryptedPassword = await bcrypt.hash(req.body.password, 10)
    const password = req.body.password
    connection.query('UPDATE users SET username = ?, email = ?, lastname = ?, firstname = ?, password = ? WHERE id =? ',[username, email, lastname, firstname ,encryptedPassword, id], (error, results) => {
        if (error) {
            res.status(401).json({err: 'la modification à échouée !' })
        }else{ 
            res.status(200).json({"Utilisateur modifié par": `${id}`}) 
        }        
    })  
    } catch(err) {
     console.error('handle the err: ', err)
    }
};

Juste un truc en l'air, ici tu utilises le package 'bcrypt'(il me semble) mais pour qu'il fonctionne tu dois installer un compiler et build the dependencies:

bcrypt is a native module for NodeJS, i.e, it uses native code to perform its work. The advantage of this over pure JS alternatives is speed. However, the disadvantage of this approach is it requires you to install a compiler and build dependencies in order to build.

vois le lien github https://github.com/kelektiv/node.bcr...n-Instructions

Pour un simple system d'authentification, il est souvent plus simple d'utiliser 'bcryptjs' https://www.npmjs.com/package/bcryptjs

[tony060]

Merci @djedjethai ca fonctionne .