È un modo sicuro per cancellare una password?

Question

La prego di dirmi se la seguente è un buon modo per hash in modo sicuro una password da memorizzare in un database:

public string CreateStrongHash(string textToHash) { 

     byte[] salt =System.Text.Encoding.ASCII.GetBytes("TeStSaLt"); 

     Rfc2898DeriveBytes k1 = new Rfc2898DeriveBytes(textToHash, salt, 1000); 
     var encryptor = SHA512.Create(); 
     var hash = encryptor.ComputeHash(k1.GetBytes(16)); 

     StringBuilder sb = new StringBuilder(); 
     for (int i = 0; i < hash.Length; i++) { 
      sb.Append(hash[i].ToString("x2")); 
     } 

     return sb.ToString(); 

    } 

Molte grazie in anticipo.

Answer 1

Si utilizza PBKDF2-SHA1, che è decente, ma non eccezionale. Bcrypt è un po 'meglio, e lo scrypt è ancora più forte. Ma poiché .net include già un'implementazione integrata in PBKDF2, questa è una scelta accettabile.

Il tuo più grande errore è che non hai ottenuto il punto di un sale. Un sale dovrebbe essere unico per ogni utente. È prassi normale creare semplicemente un valore casuale di almeno 64 bit. Memorizzalo insieme all'hash nel database.

Se si desidera, è possibile dividere il sale in due parti. Uno memorizzato nel database accanto all'utente, che è diverso per ciascuno, e una parte condivisa memorizzata altrove. Questo guadagna i vantaggi di entrambi.

Si consiglia inoltre di utilizzare un fattore di lavoro superiore a 1000. Capire quali sono le prestazioni accettabili e regolare di conseguenza. Non andrei al di sotto del 10000, e in alcune situazioni (crittografia del disco) un milione è accettabile.

Answer 2

Può essere migliorato. Per prima cosa, dovresti usare invece bcrypt. Gli hash tradizionali come SHA-512 possono essere suddivisi abbastanza facilmente con le GPU ora a un giorno. Il problema è che questi hash sono progettati per la velocità, e questo è l'opposto di ciò che si desidera in un hash della password. Bcrypt è un esempio di un algoritmo di hash adattivo. Può essere configurato in modo da richiedere un tempo "lungo" (ma comunque non causerà problemi di prestazioni nel sistema) per rendere difficile la definizione di forza bruta.

Si desidera anche rendere i sali univoci per ciascun utente.

Per ulteriori informazioni sulle modalità di hash password sicure, vedere this question.