[VB.NET] Regestrierung Per E-Mail

[AlexLPLP]

Hier ist der Code:

Button 1 /Der Sender button)

 Dim register As New MailMessage
            register.From = New MailAddress("Your E-Mail")
            register.To.Add(TextBox1.Text)
            register.Subject = "Dein Schlussel"
            register.Body = "Vielen Dank für ihre Regestrierung" & vbNewLine & vbNewLine & vbNewLine & vbNewLine & vbNewLine & "E-Mail: " & vbNewLine & TextBox1.Text & vbNewLine & vbNewLine & vbNewLine & "Danke für ihre Regestrierung!" & vbNewLine & "------------------------------------" & vbNewLine & vbNewLine & "Schlussel: " & TextBox3.Text

            Dim abschicken As New SmtpClient("smtp.web.de")
            abschicken.Port = 587
            abschicken.EnableSsl = True
            abschicken.Credentials = New Net.NetworkCredential("Your E-Mail", "Your Passwort")
            abschicken.Send(register)
            Button1.Enabled = False

Key Abfrage Button

If TextBox2.Text = TextBox3.Text Then
End If

Benötigte Tools
--------------------
3 TextBoxen
2 Labels
2 Buttons
---------------------

Bitte schaut euch noch das Video an bevor ihr es selber versucht : LINK:

Wenn ihr wollt Könnt ihr euch den Code in eine txt Downloaden, Ich habe noch ein Bild rein gepackt: LINK :

[dready]

Sofern du keine Frage dazu hast, falsches Forum gehört eher in Snippets

Dazu kommt das der Code ziemlich Chaotisch daherkommt.

1. Bau das Ding in ne Funktion mit Parametern, so das du die Variablen nicht Hardcoded drin hast und das Ding Wiederverwendbar ist

2. Da sind Rechtschreibfehler

3. Fang nicht damit an Strings in solchen Längen zu nutzen sondern Splitte sie auf mehrer Zeilen auf, ansonsten wirst du es früher oder Später bereuen, ist ziemlich nervig damit zu arbeiten

4. Dir fehlen Sanity Checks for die Daten die aus den Textboxen kommen.

5. Der SMTP Server ist Hardcoded.

6. Ändere das Naming deiner Textboxen .. so wie sie momentan benannt sind kommst du böse durcheinander, sie haben einfach keinerleih Aussagekraft.

[snow]

#moved…

[supercracker13]

Ich will dieses Script nicht schlecht reden, aber damit eine Authentifizierung zu erstellen ist zu leicht zu knacken.Mir fallen auf Anhieb mäßig Methoden ein.
1: Mit Texteditor oder hexeditor das Passwort und die mail Adresse auslesen


2: Den trafic sniffen und die mail Zugangsdaten auslesen. Bzw den Text mit dem key (Solange SSL deaktiviert ist)


3: Das Programm durch nen Debugger laufen lassen und den key rausziehen


4: eigenen SMTP hoster aufsetzten und Daten anpassen


Warum ich das schreibe: Als ich angefangen habe zu coden kam auch so etwas dabei raus und mir wurde das auch schnell klar gemacht.


Tipps: Versuch es mal über einen externen Server laufen zu lassen. Du sendest quasi dem Server die Daten und der erstellt das Passwort und versendet die mail. Damit ist das Programm schon mal sicherer. Jetzt könnte die if Abfrage nur noch gepatcht werden

[ლʕಠᴥಠʔლ]

Außerdem setzt du die benötigten root Zertifikate voraus.