De PHPhulp kroeg
layout review
excuses.
Gewijzigd op 28/07/2010 06:51:33 door Thomas de vries
*Refferal kun je niet invullen! Dat word alleen ingevuld via een refferal link!
*Als je een naam/e-mail invult en er gebeurd niks, dan is die account naam/e-mail al bezet!
Dat kan beter.
Qua lay out inderdaad.....ownee dat was niet gewenst.
Quote:
Behalve het deze zin: Nee he weer een crime game. :P
nou, ik zal het genuanceerd zeggen dan; het past goed bij het onderwerp. ;)
Nee, even serieus.
Die Welkom, , op The-Heist # zou ik wel even hetzelfde lettertype maken.
In je login klopt de form tag niet. <form method="post">
Die mist een action en mag volgens mij daar waar jij hem nu hebt neergezet ook niet.
Ook twijfel ik een beetje aan die $_GET's die jij gebruikt. Dat zou ik ook anders aanpakken.
Goed, de layout: Je maakt overal rechte hoeken, behalve bij het handboek.
Op zich vind ik die rondingen wel stoer, misschien dat je daar nog eens naar kunt kijken.
Maar bij form hoeft geen action te staan.
dit omdat het op dezelfde pagina wordt uitgevoerd.
De content is idd met ronde hoeken gedaan.
Kan natuurlijk ook kijken of ik dit met alles kan doen.
Maar dat moet ik eerst overleggen met mijn broertje.
Wij doen dit namelijk samen :D
@John
Ja die melding moet ik ook aanpassen.
Want als de gebruikersnaam wordt ingevoerd en die bestaat al dan krijgt men wel een foutmelding.
Refferal kan ik in feite verbergen en alleen te voorschijn laten komen als die ook in gebruik is.
Bedankt voor de reacties zover
Idd zie het de login page is niet W3C valid.
Zal dit gelijk in order maken.
En zal het voortaan per pagina gaan testen.
Had alleen gekeken naar de index.
Excuses
Wat ik wel vreemd vind is dat als ik de W3C laat valideren op login.php
dan maar 3 fouten. doe ik /p=login dan 10.
http://www.the-heist.cjb.net/p=index
Hele veilige server.
http://the-heist.cjb.net/cgi-bin/printenv.pl
Sja, en bestanden kun je direct openen:
http://www.the-heist.cjb.net/rechtbank.php
Gewijzigd op 27/07/2010 10:44:36 door P Lekensteyn
2) Zorg dat lettertypes een beetje gelijk zijn..
Is dat erg kan cgi wel uitzetten.
Maar sommige paginas kan alleen geopend worden als de klant ingelogt is.
De lettertypes zal ik idd ok gelijk naar kijken.
Volgens mij was het wel allemaal verdana.
Maar zal nog is kijken.
Gewijzigd op 27/07/2010 10:58:09 door thomas de vries
Denk dat je vanalles in een verkeerde volgorde aan het includen bent, of misschien dubbele html tags o.i.d. (niet verder gekeken.)
Zou eens kijken naar een frameworkje, dat heeft vaak veel voordeel.
Ook de manier van $_GET zoals ik al zei en wat Peter demonstreert zit niet zo lekker in elkaar.
alleen config.php wordt geincluded de rest niet.
wat zit met $_GET niet lekker in elkaar dan?
http://www.the-heist.cjb.net/p=index
http://www.the-heist.cjb.net/rechtbank.php
Dat is erg gevaarlijk, want zo kan ik veel met die gegevens doen.
Laat eens de code zien hoe jij die $_GET code doet.
Zoals ik al zei: ga eens naar een framework kijken, dan heb je veel van dit soort dingetjes al niet meer.
Ja daar kan ik in feite de config.php gaan includen
Zodat ie die melding niet weergeeft.
Maar goed met die melding kunnen mensen niks mee.
Maar hier is de $_GET
en dan weer iets ik heb de login.php verandert
Nu staat er alleen echo test
Maar als ik dit doe op w3c validator.
http://www.the-heist.cjb.net/p=login
Dan krijg ik foutmeldingen.
waaronder een form method.
Dit staat niet meer in de login pagina.
Even snel een simpel wat veiliger pagina systeempje gemaakt ;)
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?php
$aPaginas = array('home','nieuws','links','gastenboek','informatie','contact');
// een gewone willekeurige array met de pagina's daarin die wel vertoont mogen worden.
if(isset($_GET['p']) && trim($_GET['p']) != '')
{
if(in_array($_GET['p'],$aPaginas))
{
if(file_exists($_GET['p'].'.php'))
{
include $_GET['p'].'.php';
}
else
{
include 'home.php';}
}
else
{
include 'home.php'; }
}
?>
$aPaginas = array('home','nieuws','links','gastenboek','informatie','contact');
// een gewone willekeurige array met de pagina's daarin die wel vertoont mogen worden.
if(isset($_GET['p']) && trim($_GET['p']) != '')
{
if(in_array($_GET['p'],$aPaginas))
{
if(file_exists($_GET['p'].'.php'))
{
include $_GET['p'].'.php';
}
else
{
include 'home.php';}
}
else
{
include 'home.php'; }
}
?>
Gewijzigd op 27/07/2010 12:01:23 door Bart V B
Wat een amateurs zeg, degenen die de server onderhoud.
Je hebt mazzel dat URL fopen streams disabled zijn, anders kon de server zo overgenomen worden.
Geef me een uploadoptie en de site is hacked.
Als die URL dinges aangezet is kun je willekeurige code uitvoeren:
http://www.the-heist.cjb.net/?p=data:text/plain,<?phpinfo();?>
En het is ook nog mogelijk om te nullen.
Oplossing:
Whitelisting.
Code (php)
Dit is wel een slechte/luie manier om pagina's te weergeven.
Pagina's die niet gevonden zijn moet je met een 404 melding beantwoorden.
Anders krijg je iets als http://www.the-heist.cjb.net/p=dit-is-een-noob-site/, en wordt de homepage weergeven.
Die printenv heb ik weggehaald.
En idd goede tip een 404 meldingen te geven.
Ipv de home page als de $_GET[p] niet goed is.
Bedankt voor de tips.
Geeft een pagina zonder inhoud (maar met menu, etc.) en een 200 OK.
Als de die laatste slash achter wege laat dan krijg je wel een melding.
Maar ik ga hiermee bezig ook 404 pagina's laten genereren.
Bedankt voor de tips
Edit
Hoe kan ik ervoor zorgen dat als toch de url door de gbruiker is gewijzigd.
Ik de laatste slash ook weg laat halen.
Dus
Http://www.the-heist.cjb.net/p=ik-ben-gek
Resulteerd in pagina niet gevonden.
Maar
Http://www.the-heist.cjb.net/p=ik-ben-gek/
laat de homepage zien.
Gewijzigd op 27/07/2010 13:13:26 door thomas de vries