angular-fullstack/generator-angular-fullstack

Should encryption and hashing functions be converted to be asynchronous?

Open

#2 240 ouverte le 17 sept. 2016

Voir sur GitHub
 (0 commentaires) (1 réaction) (0 assignés)JavaScript (6 134 stars) (1 268 forks)batch import
Help Wantedenhancement

Description

I noticed the use of the crypto library implements synchronous operations. Should this be dropped for use of asynchronous? Under heavy server load this could lock up the application right?

Guide contributeur

Stack technique
javascriptnodejs
Domaine
backendperformance
Type d'issue
performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Node.jsasync/await
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Investigate the use of synchronous crypto functions in the generator's codebase. Look for `crypto.createHash` or similar calls in files that handle encryption or hashing. Propose replacing them with asynchronous counterparts using `crypto.promisify` or `require('crypto').async`. Ensure the change does not break existing functionality and consider that the generator may run synchronously during generation, so evaluate if async is feasible. Check if there are existing PRs or maintainer comments on this topic.