Leuk ! een ?page= script

[Ramon Fincken]

Leuk ! een ?page= script

Maargoed, nee daar mag dus nog wat aan gebeuren, ook zijn de tabjes van de site niet volledig leesbaar.

Ron edit: topic is een afsplitsing van http://www.webhostingtalk.nl/persone...angeboden.html

[xabin]

Leuk ! een ?page= script

Maargoed, nee daar mag dus nog wat aan gebeuren, ook zijn de tabjes van de site niet volledig leesbaar.

Wat is er mis met ?p= ??

[Ramon Fincken]

potentieel niet checken voor file exists en permissies

[Triloxigen]

En dat is belangrijk voor...?

[mikeh]

Zolang ze php in safemode draaien, no worries

[xabin]

potentieel niet checken voor file exists en permissies

Ik heb even naar zijn website gekeken, er is inderdaad geen check of de ?p= wel bestaat, dit hoort wel, maar wat dan nog, wat is er mis met ?p= ?

[Ramon Fincken]

Zolang ze php in safemode draaien, no worries

Ok, ook voor includen van backend files ?
Maargoed ik denk weer ter ver denk ik

[mikeh]

je denkt aan sql injecties ?

[®on]

Heren, dit hoort niet thuis in "personeel gezocht". Een eigen topic lijkt me meer op zijn plaats.

[Ramon Fincken]

Dank je Ron.

Maar nee, afgezien dat ik zelf de url niet mooi vind [ waarom geen mod rewrite of gewoon .html Zeker als je webdevelopment diensten aanbied .. ], zijn dit soort constructies vaak gewoon niet echt veilig omdat ze meestal een include van de GET variabele p of page doen zonder verdere checks of wat dan ook.

[ewoutz]

Hiernaast krijg je vaak een lagere rating met SEO met dat soort pagina's

[xabin]

Dank je Ron.

Maar nee, afgezien dat ik zelf de url niet mooi vind [ waarom geen mod rewrite of gewoon .html Zeker als je webdevelopment diensten aanbied .. ], zijn dit soort constructies vaak gewoon niet echt veilig omdat ze meestal een include van de GET variabele p of page doen zonder verdere checks of wat dan ook.

Dat zegt meer over jou 'script vaardigheden'.
Hoe kan jij van een scripter weten wat hij wel en wat hij niet doet?

Dat een scripter ?p= gebruikt maakt het niet onveilig. Ik vind het een erg misplaatste reactie van je.

[SF-Jeroen]

Je kan toch ook een switch maken? Met een default case?

switch($page){
case 'home':
include 'home.php';
break;
default:
include 'unknown.php';
break;
}

(Uiteraard wel eerst $page even koppelen aan de $_GET en beveiligen)

[xabin]

Je kan toch ook een switch maken? Met een default case?

switch($page){
case 'home':
include 'home.php';
break;
default:
include 'unknown.php';
break;
}

(Uiteraard wel eerst $page even koppelen aan de $_GET en beveiligen)

Zo hoort het ook.

[Ramon Fincken]

Dat zegt meer over jou 'script vaardigheden'.
Hoe kan jij van een scripter weten wat hij wel en wat hij niet doet?

Dat een scripter ?p= gebruikt maakt het niet onveilig. Ik vind het een erg misplaatste reactie van je.

En als ik zo eerlijk mag zijn vind ik dit ook geen leuke reactie. Ik heb genoeg van dat soort sites en scripts gezien en weet uit ervaring dat een boel scripters er geen of te weinig aandacht aan besteden.
Sterker nog, het kwam hier laatst nog ter sprake:
http://www.webhostingtalk.nl/1044428-post4.html
http://www.webhostingtalk.nl/1044432-post6.html

Ik heb hier een stukje code dat ik snel moest schrijven voor een simpel GET page script dat gebruik maakt van de titel in een CMS tabel:
http://www.ramonfincken.com/permalink/topic56.html
Nu is die $_GET['page'] = $row['id']; natuurlijk niet moeders mooiste stuk code, maar daar gaat het daar niet om ..

Zoals SF-Jeroen het aangeeft is de mooiste manier, maar minder practisch als je een boel bestanden hebt vanwege de hardcode stuff.