11/05/15 22:26
Goedenavond,
Heb een nieuwe hostingklant binnen gehaald.
95% van de site werkt goed, echter als ik het contactformulier invul wordt er een leeg formulier doorgestuurd naar een mailadres.
De site is www.piratenkoor.nl en dan info/boekingen.
Iemand een idee waar ik het zou moeten zoeken.
Zo ver ik kan zien is de hele site in php geprogrammeerd.
Alvast bedankt voor iemand's reactie
11/05/15 23:04
Sorry maar mijn glazenbol is vandaag net naar de garage gebracht.
Er is natuurlijk niets, maar dan ook niets over te zeggen zo. Krijg je foutmeldingen? Is de PHP versie afwijkend ten opzichte van de vorige partij enz. Zonder code is het ook maar in het wilde weg dingen roepen.
12/05/15 19:20
't is javascript + php.
En 't lijkt te werken, ook voor spammers...
12/05/15 19:35
Dat klopt maar bij versturen onthoudt hij de ingevulde waardes niet. Hij verstuurd een leeg formulier
Sent from my SM-N9005 using webhostingtalk mobile app
12/05/15 19:39
Tja dat wordt toch even je scripts nakijken of er wel wat in je post vanuit je form naar je mail script gaat. Dat is voor een ieder hier niet zo te zeggen tenzij iemand toevallig wel ze glazenbol bij de hand heeft
Verzonden vanaf mijn iPhone met behulp van webhostingtalk
13/05/15 08:51
14/05/15 11:05
Inmiddels ben ik zelf verder gekomen.
De website draait goed op PHP 4.4 en ik draai op PHP 5.5
Nu staat er in sommige bestanden $HTTP_POST_VARS deze moest ik vervangen door &$_POST;
Nu werkt het bijna.
In onderstaand bestand heb ik bovenstaande ook veranderd, echter bewaard hij de ingevulde gegevens niet.
Iemand een idee?
<?php
$title = "bestellingverstuurd";
include('header.php');
$toName = "Groot Blokzijls Piratenkoor";
$toEmail = "info@piratenkoor.nl";
$toSubject = "CD bestelling www.piratenkoor.nl";
$naam= "$initialen $achternaam";
include('common.php');
$link = dbConnect();
$query = "SELECT * FROM cds ORDER BY cdID DESC";
$result = @mysql_query($query);
$cdCount = mysql_num_rows($result);
for ($count = 0; $count < $cdCount; $count++) {
$cds = mysql_fetch_array($result);
$cdID = $cds['cdID'];
$cdName = $cds['cdName'];
$humanCount = $count+1;
$fieldValue = &$_POST[$count];
if ($fieldValue != "") {
$cdOrder .= "- $fieldValue\n";
}
}
$message = "De volgende persoon heeft een CD bestelling gedaan:\n\n"
. "Naam: $radioGroup3 $naam\n"
. "Bedrijfsnaam: $bedrijfsnaam\n"
. "Adres: $adres\n"
. "Postcode: $postcode\n"
. "Plaats: $plaats\n"
. "Land: $land\n"
. "Telefoon: $telefoon\n"
. "Gsm: $gsm\n"
. "Fax: $fax\n"
. "Emailadres: $emailadres\n\n"
. "Deze persoon wil graag de volgende CD(s):\n"
. "$cdOrder\n\n";
mail($toName." <".$toEmail.">",$toSubject, $message, "From: ".$naam." <".$emailadres.">");
$cdOrder2 = nl2br($cdOrder);
$message2 = "Uw bestelling is succesvol verstuurd. U ontvangt uw bestelling per post.<br><br>\n\n"
. "Hier volgt een overzicht van de informatie die u heeft verstuurd:<br><br>\n\n"
. "Naam: $radioGroup3 $naam<br>\n"
. "Bedrijfsnaam: $bedrijfsnaam<br>\n"
. "Adres: $adres<br>\n"
. "Postcode: $postcode<br>\n"
. "Plaats: $plaats<br>\n"
. "Land: $land<br>\n"
. "Telefoon: $telefoon<br>\n"
. "Gsm: $gsm<br>\n"
. "Fax: $fax<br>\n"
. "Emailadres: $emailadres<br><br>\n\n"
. "U bestelde de volgende CD(s):<br>\n"
. "$cdOrder2<br><br>\n\n";
print "
<table width=\"100%\" border=\"0\" cellpadding=\"0\" cellspacing=\"0\">
<tr>
<td width=\"100%\" align=\"left\" valign=\"top\">$message2</td>
</tr>
</table>
";
include('footer.php');
?>
14/05/15 11:38
Bij een formulier wordt meestal gebruik gemaakt van php $_POST. Bij oudere PHP versies kon register_globals worden gemaakt om simpelweg $veldnaam te gebruiken om verzonden data op te halen en te hergebruiken (bijv. in een echo). PHP 5.4 en hoger ondersteunt register_globals niet meer waardoor deze methode niet meer werkt. Als je een formulier post dan moet je de velden ophalen met $_POST['veldnaam']. Bijv. $naam in je script zou eerst moeten worden gedefinieerd, bijv. $naam = $_POST['naam']; (waarbij naam gelijk is aan de naam van het formulier veld).
Als je het veilig wilt doen gebruik je bijv. htmlspecialchars en stripslashes om de opgehaalde $_POST var te strippen van schadelijke code.
14/05/15 13:23
Haal ook de @ voor mysql_query weg. Fouten onderdruk je niet maar hoor je netjes op te vangen. Als er nu iets fout gaat mis je output.
Sent from my GT-I9505 using webhostingtalk mobile app
15/05/15 08:04
Gezien van wat ik hier zie .. neem een google embedded form of mailchimp form. Gratis, veiliger, sneller en je hebt een goede lijst met kopers/geinteresseerden.
Klarned Managed hosting Hosting, optimalisatie & debugging door WP Core developers
15/05/15 09:04
Heb het aangepast naar de opmerkingen van de mensen hier in het topic voor je.
Ook de for loop even vervangen voor een while. + de mysql error netjes afgevangen.
[update]
Ook de post variablen toch maar even aangepast gelijk voor je
Dit laat het formulier werken, het maakt het niet veilig, voor injecties,crossite etc.
PHP Code:<?php
$title = "bestellingverstuurd";
include('header.php');
$toName = "Groot Blokzijls Piratenkoor";
$toEmail = "info@piratenkoor.nl";
$toSubject = "CD bestelling www.piratenkoor.nl";
$naam= "$initialen $achternaam";
include('common.php');
$link = dbConnect();
$query = "SELECT * FROM cds ORDER BY cdID DESC";
// Kan je natuurlijk ook netjes afvangen in een if controle, maar een die voldoet even
$result = mysql_query($query) or die(mysql_error());
$cdCount = mysql_num_rows($result);
$counter = 0;
$sCDOrder = '';
while($aCDInfo = mysql_fetch_array($result))
{
$iCDID = $aCDInfo['cdID'];
$sCDName = $aCDInfo['cdName'];
if(isset($_POST[$count]))
{
$cdOrder .= "- ".$_POST['count']."";
}
$count++;
}
$message = "De volgende persoon heeft een CD bestelling gedaan:\n\n"
. "Naam: ".$_POST['radioGroup3']." ".$_POST['$naam']."\n"
. "Bedrijfsnaam: ".$_POST['bedrijfsnaam']."\n"
. "Adres: ".$_POST['adres']."\n"
. "Postcode: ".$_POST['postcode']."\n"
. "Plaats: ".$_POST['plaats']."\n"
. "Land: ".$_POST['land']."\n"
. "Telefoon: ".$_POST['telefoon']."\n"
. "Gsm: ".$_POST['gsm']."\n"
. "Fax: ".$_POST['fax']."\n"
. "Emailadres: ".$_POST['emailadres']."\n\n"
. "Deze persoon wil graag de volgende CD(s):\n"
. "$cdOrder\n\n";
mail($toName." <".$toEmail.">",$toSubject, $message, "From: ".$_POST['naam']." <".$_POST['emailadres'].">");
$cdOrder2 = nl2br($cdOrder);
$message2 = "Uw bestelling is succesvol verstuurd. U ontvangt uw bestelling per post.<br><br>\n\n"
. "Hier volgt een overzicht van de informatie die u heeft verstuurd:<br><br>\n\n"
. "Naam: ".$_POST['radioGroup3']." ".$_POST['naam']."<br>\n"
. "Bedrijfsnaam: ".$_POST['bedrijfsnaam']."<br>\n"
. "Adres: ".$_POST['adres']."<br>\n"
. "Postcode: ".$_POST['postcode']."<br>\n"
. "Plaats: ".$_POST['plaats']."<br>\n"
. "Land: ".$_POST['land']."<br>\n"
. "Telefoon: ".$_POST['telefoon']."<br>\n"
. "Gsm: ".$_POST['gsm']."<br>\n"
. "Fax: ".$_POST['fax']."<br>\n"
. "Emailadres: ".$_POST['emailadres']."<br><br>\n\n"
. "U bestelde de volgende CD(s):<br>\n"
. "".$_POST['cdOrder2']."<br><br>\n\n";
print "
<table width=\"100%\" border=\"0\" cellpadding=\"0\" cellspacing=\"0\">
<tr>
<td width=\"100%\" align=\"left\" valign=\"top\">$message2</td>
</tr>
</table>
";
include('footer.php');
?>
Laatst gewijzigd door MarkKapitein; 15/05/15 om 09:10.
15/05/15 09:49
ZoDe website draait goed op PHP 4.4 en ik draai op PHP 5.5ook goeie morgen, PHP 4.4? Released: 07 August 2008 en intussen zijn er geen updates meer geweest voor deze versie!!
Deze server was niet alleen onveilig, hij was zelfs zo lek als een vergiet zonder bodem.
In oude PHP versies zitten zitten een paar flinke veiligheidslekken, waaronder één met de mail() functie waarmee het mogelijk is om dit te misbruiken voor het versturen van spam berichten. http://foundationphp.com/blog/2010/1...der-injection/
Ok terug naar het script.
* mysql_fetch_array(): Gebruik liever mysql_fetch_assoc(), deze kost minder geheugen
* $cdOrder .= "- ".$_POST['count'].""; (die ."" is nergens voor nodig. en volgens mij klopt hier iets niet
15/05/15 10:18
Maar jij moet de code van de klant gaan aanpassen om op jouw hostingservers te werken? Ligt deze verantwoording niet bij je klant?
Marin Heideman (DigiState B.V.)
15/05/15 10:19
@Marin ,
Ik ga uit van niet, wordt anders wel een heel leuk boeltje bij iedereen.
Ik gok dat deze persoon in kwestie zijn nieuwe klant wil helpen om alles te migreren en daar ook support probeert bij te leveren
@SebastiaanStok
Bedankt voor de info, Werk zelf alleen maar met laravel, dit was ook even weer opfrissen
Laatst gewijzigd door MarkKapitein; 15/05/15 om 10:21.
Likes: 
Quote
Oorspronkelijk geplaatst door DB Service
