Math.random() is not cryptographically secure
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- javascript, nodejs
- Domaine
- authentication, security
Piste de recherche
Commencez dans lib/utils.js à la ligne 61, où Math.random() est utilisé pour générer des UIDs. Consultez la documentation liée de Node.js crypto.randomInt et vérifiez comment ces UIDs sont utilisés dans toute la bibliothèque SAML. Le travail est terminé lorsque la génération des UIDs ne repose plus sur Math.random() et préserve le comportement d’unicité de la bibliothèque.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
https://github.com/auth0/node-saml/blob/7ee79849d6b0a0935e42ad456ae8cb92d2b3bb93/lib/utils.js#L61
This function is used to generate unique UIDs throughout the SAML library. Given that SAML deals with authentication, it seems like poor practice to not generate random UIDs in a cryptographically secure manner.
Reference: https://nodejs.org/api/crypto.html#crypto_crypto_randomint_min_max_callback
Guidance: https://gist.github.com/joepie91/7105003c3b26e65efcea63f3db82dfba
- Langage dominant
- JavaScript
- Étoiles
- 73
- Forks
- 89
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
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.
Autres issues de auth0/node-saml
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 10/100
-
Incompatibility with @xmldom/xmldom 0.9.x: DOMParser.parseFromString throws on undefined mimeTypePeut-être pris @esetnik l’a pris il y a 177 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
Toutes les issues de auth0/node-saml
Issues similaires
-
赞~Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
DeanChensj/jiangshan-map#221 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 13 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
AtlasOfLivingAustralia/species-lists#874 ·
Les mainteneurs répondent en général sous 1 jour
-
Slack: DM thread replies sent with "Also send to..." never reach the agentPeut-être pris @roboeve l’a pris aujourd’hui. Ouvertebug channel:slack p2
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
vercel/eve#5774 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour