gistfile1.cs

Public Class HashLogin

	Public Shared Function Run(email As String, salt As String) As String
		Dim md5Hasher As MD5 = MD5.Create()
		Dim data As Byte() = md5Hasher.ComputeHash(Encoding.[Default].GetBytes(password & salt))

		Dim stringBuilder = New StringBuilder()

		For i As Integer = 0 To data.Length - 1
			stringBuilder.Append(data(i).ToString("x2"))
		Next


		Return stringBuilder.ToString()

	End Function
End Class

Warum zur Hölle werden Hashs eigentlich immer in Strings umgewandelt? man kann doch direkt das Byte-Array speichern.

und wie c# sieht das nicht aus.

Und man könnte direkt nach Base64 kodieren.

public class Hashing : IHashing
{
    public string GetHash(string text, string salt)
    {
        var hasher = SHA1.Create();
        return Convert.ToBase64String(hasher.ComputeHash(Encoding.UTF8.GetBytes(text + salt)));
    }
}

Hallo Albert, das wurde mühselig aus einem anderen Projekt in VB.NET konvertiert ;-) Danke für die Base64 Optimierung! MD5 ist ein bisschen kürzer und für den Anwendungsfall als Einmal-Login etwas geeigneter.

(Das IHashing Interface geht mir einen Tick zu weit, auch für IoC würde ich direkt den Typen registrieren.)

IHashing weil ich wohl verschiedenen Hasher einbauen muss, wegen Rückwärtskompabilität.

Desweiteren ist es einfacher ein IHashing mit Hashing zu registrieren (per Konvention) als bestimmte Klassen einzeln zu registrieren. Deshalb Interfacerities ;)