WinFuture-Forum.de: Kontaktformular Versagt -.- - WinFuture-Forum.de

Zum Inhalt wechseln

Nachrichten zum Thema: Entwicklung
Seite 1 von 1

Kontaktformular Versagt -.- kann auch am server liegen -.-


#1 Mitglied ist offline   KlacM 

  • Gruppe: aktive Mitglieder
  • Beiträge: 29
  • Beigetreten: 23. März 05
  • Reputation: 0

geschrieben 10. Juli 2005 - 20:13

Hallo, mein selbst gebastelter Kontaktmailer will nicht mailen. es steht zwar da, dass die mail gesendet wurde, aber ich erhalte keine mail.

link zum mailer: http://web486.srv01....test/mailer.php

im anhang is der mailer.

kann natürlich auch sein, dass der serveradmin den mailserver deaktiviert hat

danke im vorraus für die hilfe

gruß klacm

Angehängte Datei(en)


0

Anzeige



#2 Mitglied ist offline   Meatwad 

  • Gruppe: aktive Mitglieder
  • Beiträge: 784
  • Beigetreten: 07. August 04
  • Reputation: 0
  • Geschlecht:Männlich
  • Wohnort:Bad Vilbel
  • Interessen:Computer, Inliner, Chatten...

geschrieben 10. Juli 2005 - 20:59

hmmm...das ding ist, sorry, totalschlecht gecoded. du verwendest die $_POST-variable nicht, ..warscheinlich ist register_globals off! das kann nicht funzen...ich hab mir da mal ein script geschrieben. das tuhe ich mals als anhang machen

Angehängte Datei  email.php (984bytes)
Anzahl der Downloads: 29

Dieser Beitrag wurde von Meatwad bearbeitet: 10. Juli 2005 - 21:03

I am away to look for my self. If I am back before I return keep me here. | Mein Server
0

#3 Mitglied ist offline   Floele 

  • Gruppe: aktive Mitglieder
  • Beiträge: 919
  • Beigetreten: 22. Juni 04
  • Reputation: 0

geschrieben 10. Juli 2005 - 21:11

Wenn du schon andere kritisierst, dann mach es wenigstens selber richtig :)
Das hier
$_POST[subject]
ist nämlich ein ziemlich schlechter Stil.

Dieser Beitrag wurde von Floele bearbeitet: 10. Juli 2005 - 21:11

0

#4 Mitglied ist offline   KlacM 

  • Gruppe: aktive Mitglieder
  • Beiträge: 29
  • Beigetreten: 23. März 05
  • Reputation: 0

geschrieben 10. Juli 2005 - 21:18

@meatwad ob es total schlecht gecoded ist, ist mir egal, da ich erstens kein php pro bin(eher php-noob) und zweitens es nur die aufgaben eines kontaktformulars erfüllen soll

edit: es lag nur am server. ich hab es mal auf einen anderen Server geladen(in dem Fall funpic) und ich hab die testmails bekommen. also das Thema ist geklärt.

@MeatWad ich benutz nicht dein script. ich will mal selber in die gänge kommen, und nicht immer auf andere angewiesen sein müssen, deswegen benutz ich mein script.
trotzdem danke für die Hilfe
Gruß KlacM

Dieser Beitrag wurde von KlacM bearbeitet: 11. Juli 2005 - 15:49

0

#5 Mitglied ist offline   Meatwad 

  • Gruppe: aktive Mitglieder
  • Beiträge: 784
  • Beigetreten: 07. August 04
  • Reputation: 0
  • Geschlecht:Männlich
  • Wohnort:Bad Vilbel
  • Interessen:Computer, Inliner, Chatten...

geschrieben 11. Juli 2005 - 10:54

@Floele: ja, ich verwende keine ' in arraynamen...stimmt...aber was daran so schlimm?

@KlacM: ok, viel glück
I am away to look for my self. If I am back before I return keep me here. | Mein Server
0

#6 Mitglied ist offline   Slayer 

  • Gruppe: aktive Mitglieder
  • Beiträge: 1.476
  • Beigetreten: 12. Dezember 03
  • Reputation: 0
  • Geschlecht:Männlich
  • Wohnort:Baden-Württemberg
  • Interessen:&gt; Internet<br />&gt; Musik<br />&gt; Filme<br />&gt; Events

geschrieben 11. Juli 2005 - 14:12

So, ich habe das jetzt mal umgeschrieben und nun müsste es gehen, habe es aber nicht getestet.
Da waren ja massig Fehler drin, das konnte gar nicht funktionieren ;D

Aber es ist immer noch nicht schön formatiert, das musst du selber regeln!

Die Dateien sind im Anhang, viel Spaß damit :D

Angehängte Datei(en)


Dieser Beitrag wurde von Slayer bearbeitet: 11. Juli 2005 - 14:12

Der Mensch hat drei Wege, klug zu handeln.
Erstens durch Nachdenken: Das ist der Edelste.
Zweitens durch Nachahmen: Das ist der Leichteste.
Drittens durch Erfahrung: Das ist der Bitterste.

(Konfuzius)
0

#7 Mitglied ist offline   KlacM 

  • Gruppe: aktive Mitglieder
  • Beiträge: 29
  • Beigetreten: 23. März 05
  • Reputation: 0

geschrieben 11. Juli 2005 - 15:48

fehler? eigentlich dürfte nur wärend des headers .= zeug fehler drin gewesen sein(wenn schon), weil die grundfunktion auf dem mail() php tag liegt, also was kann man da noch so alles falsch machen? ausserdem lies mal meinen editierten thread
0

#8 Mitglied ist offline   Slayer 

  • Gruppe: aktive Mitglieder
  • Beiträge: 1.476
  • Beigetreten: 12. Dezember 03
  • Reputation: 0
  • Geschlecht:Männlich
  • Wohnort:Baden-Württemberg
  • Interessen:&gt; Internet<br />&gt; Musik<br />&gt; Filme<br />&gt; Events

geschrieben 11. Juli 2005 - 19:21

Zitat (KlacM: 11.07.2005, 16:48)

fehler? eigentlich dürfte nur wärend des headers .= zeug fehler drin gewesen sein(wenn schon), weil die grundfunktion auf dem mail() php tag liegt, also was kann man da noch so alles falsch machen? ausserdem lies mal meinen editierten thread
<{POST_SNAPBACK}>

Ja, ich wollte dir nur helfen.
Schaue dir meine Dateien an, dann siehst du, was du midnestens besser machen kannst.
Doch, da waren genug Fehler drin. z.B. keine Post-Variablen etc.
Der Mensch hat drei Wege, klug zu handeln.
Erstens durch Nachdenken: Das ist der Edelste.
Zweitens durch Nachahmen: Das ist der Leichteste.
Drittens durch Erfahrung: Das ist der Bitterste.

(Konfuzius)
0

#9 Mitglied ist offline   KlacM 

  • Gruppe: aktive Mitglieder
  • Beiträge: 29
  • Beigetreten: 23. März 05
  • Reputation: 0

geschrieben 11. Juli 2005 - 20:44

jemand aus dem netz sagte zu mir, dass kein fehler drin sei, aber der codingsyntax etwas veraltet sei. und ich sehe veraltet nicht als fehler an sondern als genauso gut, aber anders
0

#10 Mitglied ist offline   Floele 

  • Gruppe: aktive Mitglieder
  • Beiträge: 919
  • Beigetreten: 22. Juni 04
  • Reputation: 0

geschrieben 11. Juli 2005 - 21:44

Zitat (Meatwad: 11.07.2005, 11:54)

@Floele: ja, ich verwende keine ' in arraynamen...stimmt...aber was daran so schlimm?


Dass es sich dann eigentlich um Konstanten handelt. Ich finde jetzt leider keine Seite wo das näher erklärt wird, aber es ist eigentlich falsch wie du es schreibst. Slayer musste ich das auch schon erklären ;)

Zitat (Slayer: 11.07.2005, 15:12)

So, ich habe das jetzt mal umgeschrieben und nun müsste es gehen, habe es aber nicht getestet.
Da waren ja massig Fehler drin, das konnte gar nicht funktionieren ;)


Wenn man die Sicherheit betrachtet dürfte da immer noch ein schicker Fehler drin sein ;)

Zitat

jemand aus dem netz sagte zu mir, dass kein fehler drin sei, aber der codingsyntax etwas veraltet sei. und ich sehe veraltet nicht als fehler an sondern als genauso gut, aber anders
<{POST_SNAPBACK}>


Veraltete Scripts sind unter Umständen (in diesem Fall mit Sicherheit) definitiv weniger brauchbar als "aktuelle". Außerdem erzeugt der veraltete Kram keineswegs vertrauen in das Skript.
0

#11 Mitglied ist offline   Meatwad 

  • Gruppe: aktive Mitglieder
  • Beiträge: 784
  • Beigetreten: 07. August 04
  • Reputation: 0
  • Geschlecht:Männlich
  • Wohnort:Bad Vilbel
  • Interessen:Computer, Inliner, Chatten...

geschrieben 12. Juli 2005 - 11:19

@floele: stimmt, da hast du recht, das wären dan konstanten...ich werde mich mal umgewöhnen müssen ;-)
I am away to look for my self. If I am back before I return keep me here. | Mein Server
0

#12 Mitglied ist offline   Slayer 

  • Gruppe: aktive Mitglieder
  • Beiträge: 1.476
  • Beigetreten: 12. Dezember 03
  • Reputation: 0
  • Geschlecht:Männlich
  • Wohnort:Baden-Württemberg
  • Interessen:&gt; Internet<br />&gt; Musik<br />&gt; Filme<br />&gt; Events

geschrieben 12. Juli 2005 - 14:35

Doch, da waren auch Fehler drin, glaube es mir doch.

@Floele
Welcher Fehler wäre dies, das kann schon mal passiert sein :wallbash:
Der Mensch hat drei Wege, klug zu handeln.
Erstens durch Nachdenken: Das ist der Edelste.
Zweitens durch Nachahmen: Das ist der Leichteste.
Drittens durch Erfahrung: Das ist der Bitterste.

(Konfuzius)
0

#13 Mitglied ist offline   Floele 

  • Gruppe: aktive Mitglieder
  • Beiträge: 919
  • Beigetreten: 22. Juni 04
  • Reputation: 0

geschrieben 12. Juli 2005 - 15:32

Der Fehler liegt hier:

$absender = $_POST['Absender'];
[...]
mail($empfaenger,$betreff,$mailinhalt,"From: ".$absender." <".$absender.">");


Du überprüfst nicht ob der Absender eine gültige E-Mail Adresse ist. Folglich kann man durch bestimmte Absenderangaben die Zeichenkette für die zusätzlichen E-Mail Headers ähnlich wie bei einer SQL Injektion manipulieren. Im Prinzip könnte man solche lückenhaften E-Mail Formulare dazu benutzen E-Mails von deinem Server aus an irgendwelche Leute zu schicken , ich weiß aber nicht inwiefern man durch das " <".$absender.">" daran gehindert wird. Jedenfalls musst du aufpassen wenn du vom Benutzer gesendete Variablen in diese Header reinsteckst.

Dieser Beitrag wurde von Floele bearbeitet: 12. Juli 2005 - 15:32

0

#14 Mitglied ist offline   Slayer 

  • Gruppe: aktive Mitglieder
  • Beiträge: 1.476
  • Beigetreten: 12. Dezember 03
  • Reputation: 0
  • Geschlecht:Männlich
  • Wohnort:Baden-Württemberg
  • Interessen:&gt; Internet<br />&gt; Musik<br />&gt; Filme<br />&gt; Events

geschrieben 12. Juli 2005 - 15:37

Da hast du Recht, das hatte ich vergessen, die Mail sollte auch auf die Richtigkeit überprüft werden.
Das kann man ja mit dieser Funktion üerprüfen:

function check_mail($email)
{
	$pattern = "/^[\w\d]+([\.-]?[\w\d]+)*@\w+([\.-]?[\w\d]+)*\.\w{2}\w*$/i";
	if(preg_match($pattern, $email)) return TRUE; else return FALSE;
}

Der Mensch hat drei Wege, klug zu handeln.
Erstens durch Nachdenken: Das ist der Edelste.
Zweitens durch Nachahmen: Das ist der Leichteste.
Drittens durch Erfahrung: Das ist der Bitterste.

(Konfuzius)
0

Thema verteilen:


Seite 1 von 1

1 Besucher lesen dieses Thema
Mitglieder: 0, Gäste: 1, unsichtbare Mitglieder: 0