Blokowanie dostępu do strony, sprawdzenie poprawności kodu... Opcje

[Forgoot]

Witam!
Jako że jestem początkujący proszę Was, abyście powiedzieli czy ten skrypt jest poprawnie napisany, jeśli nie to proszę o ewentualną poprawkę i komentarz "co źle" zrobilem... Może da się ten skrypt rozbudować, dodać "coś", jeśli tak to proszę o podpowiedzi...

[PHP] pobierz, plaintext 
<?php
/**************************************************************************
 *  Prosty skrypt zabraniający dostępu do strony określonemu adresowi IP  *
 *  zapisanego w pliku ban.txt.                                           *
 **************************************************************************/
 
$adres = strval($_SERVER['REMOTE_ADDR']); 
$ip_zbanowane = @file("ban.txt"); 
    if($_SERVER['REMOTE_ADDR'] == "$ip_zbanowane")
  {
          /*Kod strony*/ 
    }else{
echo("<b>ERROR!</b> Nie masz uprawnień do oglądania tej strony.<br /> Twój IP: $adres został zbanowany.");
  }
 
?>
[PHP] pobierz, plaintext 

Plik ban.txt wygląda (przykładowo) tak:

Kod

127.0.0.1
127.0.0.2
127.0.0.3
127.0.0.4

Pozdrawiam.

(IMG:http://forum.php.pl/uwaga.gif)
~mike_mech

Ten post edytował Forgoot 27.10.2005, 10:11:57

[mike]

Funkcja file() zwraca tablicę, co oznacza, że ta linia:

[PHP] pobierz, plaintext 
<?php
 
//...
if($_SERVER['REMOTE_ADDR'] == "$ip_zbanowane")
//...
 
?>
[PHP] pobierz, plaintext 

Jest błędna bo porównujesz stringa do tablicy i to jeszcze źle to robisz (po co Ci znaki cudzysłowiu ).
Lepiej będzie:

[PHP] pobierz, plaintext 
<?php
 
//...
if( in_array( $_SERVER['REMOTE_ADDR'], $ip_zbanowane ) )
//...
 
?>
[PHP] pobierz, plaintext 

Teraz php sprawdzi czy adres usera jest w tablicy IP zbanowanych userów.

P.S. $_SERVER może zawierać mało wiarygodny adres IP, aby mieć większą pewnośc co do adresu, użyj funkcji:

[PHP] pobierz, plaintext 
<?php
function GetClientIP()
{
	$ip = 0;
 
	if( ! empty( $_SERVER[ 'HTTP_CLIENT_IP' ] ) )
	{
		$ip = $_SERVER[ 'HTTP_CLIENT_IP' ];
	}
 
	if( ! empty( $_SERVER[ 'HTTP_X_FORWARDED_FOR' ] ) )
	{
		$ipList = explode( ", ", $_SERVER[ 'HTTP_X_FORWARDED_FOR' ] );
		if( $ip )
		{
			array_unshift( $ipList, $ip );
			$ip = 0;
		}
		foreach( $ipList as $v )
		{
			if( ! eregi( "^(192.168|172.16|10|224|240|127|0.", $v ) )
			{
				return $v;
			}
		}
	}
 
	return $ip ? $ip : $_SERVER[ 'REMOTE_ADR' ];
}
 
?>
[PHP] pobierz, plaintext