[php] IP Ban script werkt niet

[Pronk]

Ik heb een IP Ban scriptje gebouwd, het IP blok gedeelte werkt goed maar om een of andere duistere rede werkt het "ELSE" gedeelte niet. Als het IP adres van de gebruiker niet in de database staat wordt "forum.php" dus niet geladen.

Ziet iemand wat ik fout doe?

PHP:

<?   
$dbhost = "#####";   
$dbuser = "#####";   
$dbpass = "#####";   
$dbname = "#####";   
mysql_connect($dbhost,$dbuser,$dbpass) or die("Ik ben dood"); 
mysql_select_db ($dbname);   
$font = "arial";   
$fontsize = "2";   
$ip = $_SERVER['REMOTE_ADDR'];
$banbericht = "Jij mag er lekker niet in!";   

$query = mysql_query("SELECT * FROM forumblok");   
$result = mysql_query($query); 
while($object = mysql_fetch_object($result)){  
    if ($ip == $object->ip) {   
    echo "<font face=$font size=$fontsize><b>$banbericht</b></font>"; 
    break;
  }else{   
    header("Location: forum.php"); 
    break; 
  } 
} 
?>

[ Voor 34% gewijzigd door Pronk op 18-11-2003 20:19 ]

[Osiris]

Huh? 2x achter elkaar mysql_query? Dat lijkt me niet gezond..

Over t algemeen werk je met:

[php]
$query = "SELECT * FROM Weetikveelwat";
$result = mysql_query($query);
While ($row = mysql_fetch_array($result)
{

print_r($row);

}
[/]

OK, dat had je al door, maar dan snap ik alsnog je dubbele mysql_query niet, aangezien die functie geen flikker met een query-result kan, toch?

[ Voor 26% gewijzigd door Osiris op 18-11-2003 20:21 ]

[gorgi_19]

Een beetje offtopic, maar waarom doe je niet:

SQL:

Select Count(IP) as AantalBans FROM forumblok WHERE IP = ipadress

[Willem2]

Volgens mij spring je met die break niet uit je while loop maar uit je if statement.

Je kunt het beter zo doen denk :

PHP:

$query = "SELECT ip FROM forumblok WHERE ip='$ip'"; 
$result = mysql_num_rows (mysql_query ($query));

if ( $result <> 0 ) echo 'je mag er niet in';
else include 'forum.php);

Of iets in die trant (ben niet zo'n mysql held)

[ Voor 19% gewijzigd door Willem2 op 18-11-2003 20:23 . Reden: /spuit 11. bovenstaande is mooier! :) ]

[GlowMouse]

Er zijn ook browsers die niet overweg kunnen met je header. Internet Explorer zal er geen problemen mee hebben, maar omwille van de compatibiliteit kun je beter het volledige pad inclusief het protocol geven.

Een WHERE clause in je query is inderdaad aan te raden.

Wat gebeurt er als mensen gelijk naar forum.php gaan? Hele beveiliging naar de soep?

[Willem2]

header loopt niet altijd even lekker.

Deze doet het *altijd* (voor zover ik heb meegemaakt):

<META HTTP-EQUIV="refresh" content="0; URL=forum.php">

[Nobby]

GlowMouse schreef op 18 november 2003 @ 20:24:
Wat gebeurt er als mensen gelijk naar forum.php gaan? Hele beveiliging naar de soep?

Dat is wel op te lossen door headers te checken

[GlowMouse]

Willem2 schreef op 18 november 2003 @ 20:25:
header loopt niet altijd even lekker.

Bij mij loopt header wel altijd lekker

Je SQL post is minder goed als die van je voorganger, dat van je voorganger is namelijk efficienter.

DAlchemist schreef op 18 november 2003 @ 20:26:
Dat is wel op te lossen door headers te checken

Referers zijn te faken.....

[ Voor 27% gewijzigd door GlowMouse op 18-11-2003 20:27 . Reden: reactie op DAlchemist toegevoegd ]

[Willem2]

GlowMouse schreef op 18 november 2003 @ 20:27:
[...]


Bij mij loopt header wel altijd lekker

Je SQL post is minder goed als die van je voorganger, dat van je voorganger is namelijk efficienter.

Nou, bepaalde browsers vreten die header als redirect echt niet... Maar dat is een beetje offtopic hier!

en dat mijn SQL post niet zo proper is; dat had ik zelf ook al aangegeven...

(dus niet)

[ Voor 2% gewijzigd door Willem2 op 18-11-2003 20:30 . Reden: geheugen faalt. Tijd voor een ramtest... :) ]

[gorgi_19]

Om het geheel trouwens nog efficienter te maken, kan je ook je IP omzetten naar een Long en deze opslaan in de database.

[marty]

je hoort na die header altijd een exit te doen. van www.php.net:

PHP:

header("Location: http://www.example.com/"); /* Redirect browser */

/* Make sure that code below does not get executed when we redirect. */
exit;

[Pronk]

Oke mensen ik snap het nu en het werkt. thx!

[Jurgle]

Referers zijn inderdaad te faken, headers worden niet door alle browsers ondersteund en dit laatste geldt ook voor de <META> HTML tag.

[.oisyn]

taal aan titel toegevoegd

[Soultaker]

Ah, weer een ouderwets iedereen-blaat-maar-raak topic.

Willem2 schreef op 18 november 2003 @ 20:29:
Nou, bepaalde browsers vreten die header als redirect echt niet...

Noem er eens een paar dan?

[.oisyn]

Jurgle schreef op 18 november 2003 @ 22:03:
headers worden niet door alle browsers ondersteund

headers worden sowieso ondersteund, anders valt er weinig te communiceren. Je hebt het waarschijnlijk over de redirect. Volgens de specs moet dat een volledige uri zijn. Dus niet forum.php of /mijnpad/blaat/forum.php, maar [i][norml]http://www.mijndomein.nl/mijnpad/blaat/forum.php[/][/]

Content na een redirect header is niet valid. Webservers kunnen dat overigens afvangen, ik weet van IIS dat als je geen volledige uri meegeeft, of als je output genereert na een redirect header, dat ie dan intern redirect, en het voor de browser dan niet opvalt

Willem2 schreef op 18 november 2003 @ 20:22:
Volgens mij spring je met die break niet uit je while loop maar uit je if statement.

met break spring je uit loop statements (for, while, do) en een switch statement. Je kunt er niet puur mee uit een if springen.

[dusty]

Zoals ik het zie wordt de eerste IP gecontroleerd.. Hey die is niet gebanned -> Forwarden naar het forum.php

Dus totaal kan je altijd maar een IP bannen ? beetje veel moeite zo'n heel systeem voor slechts een IP

[Bor]

Verder kom je er met een proxy zo doorheen met dit script. Check php.net daar staan voorbeelden van hoe je zo goed mogelijk het ip achter de proxy kan achterhalen.