node-red / node-red/node-red-admin
[secuity]The iteration count for the salt is too low
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 51
- Forks
- 39
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
At the location https://github.com/node-red/node-red-admin/blob/master/lib/commands/hash.js#L27, the encryption of the salt with bcrypt.hashSync(password, 8) should have a loop count of at least 10 to ensure sufficient security.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Ouvrez lib/commands/hash.js à la ligne 27 et examinez l’appel à bcrypt.hashSync. Mettez à jour le nombre d’itérations du salt à au moins 10, puis exécutez les vérifications disponibles dans le dépôt pour confirmer que la commande de hachage fonctionne toujours avec le paramètre modifié.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- security
- Type d'issue
- Bug
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 45/100