Forum PHP.pl

Mam taki kod, co należy zrobić żeby można było banować kilka adresów IP, a nie tylko 1? Bardzo proszę o pomoc.

[PHP] pobierz, plaintext 
<?php
$userip = $HTTP_SERVER_VARS['REMOTE_ADDR']; 
$banip = "11.11.11.11"; 
if ($userip==$banip) 
{ 
echo 'Twoje IP zostało zablokowane'; 
exit; 
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$userip = $_SERVER['REMOTE_ADDR']; 
$banip =array('11.11.11.11','27.27.27.27'); 
 
foreach( $banip as $ip )
{
  if ($userip==$ip) 
  { 
	echo 'Twoje IP zostało zablokowane'; 
	exit; 
  }
}
?>
[PHP] pobierz, plaintext 

moze nie dzialac (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) z pamieci napisalem to (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) .

pozdrawiam

ps.
odpowiedz na twoje pytanie to : "przerobic skrypt" (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Zrób tablicę ban_ip=array('127.0.0.1','192.168.1.1'); itd....
a potem sprawdzaj
if (in_array($userip,$ip) ){
//jest w tablicy czyli zabanowany
} else {
//nie ma go w tablicy
}

Dziękuję (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)
Mam teraz tak:

[PHP] pobierz, plaintext 
<?php
$userip = $HTTP_SERVER_VARS['REMOTE_ADDR']; 
$banip=array('11.11.11.11', '123.132.123');
if (in_array($userip,$banip))
{ 
echo 'Twoje IP zostało zablokowane'; 
exit; 
}
?>
[PHP] pobierz, plaintext 

Działa (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

A teraz kolejna rzecz...
Jak zrobić, aby adresy ip były pobierane z pliku lub z bazy danych?

Ten post edytował terreska 13.01.2007, 12:13:29

A zrób tak jak napisałem. Będzie o wiele szybciej i będzie mniej kodu

Już właśnie tak zrobiłam (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif) i liczę na dalszą pomoc (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Plik ipeki.txt (każdy IP w oddzielnej lini)


Zmiany w kodzie to

[PHP] pobierz, plaintext 
<?php
//zamiana
$banip=array('11.11.11.11', '123.132.123');
//na
$banip=file("ipeki.txt");
?>
[PHP] pobierz, plaintext 

Pozdrawiam.

Ta sama zmiana chyba nie wystarczy.
Co jeszcze oprócz tego zmienić?

Owszem, wystarczy, chyba, że inne rzeczy zepsułaś : ) (do funkcji file() musisz podać nazwę pliku razem ze ścieżką)

Pozdrawiam.

A co z tym:

[PHP] pobierz, plaintext 
<?php
if (in_array($userip,$banip))
?>
[PHP] pobierz, plaintext 

?

... no bo mi nie działa, tzn. żaden błąd nie wyskakuje, ale i nie banuje.

Tzn jak nie banuje?
Zamiast HTTP_SERVER_VARS uzywaj $_SERVER["REMOTE_ADDR"];.
Zresztą - odrobina samodzielności nie zawadzi prawda?

Pozdrawiam.

No nie banuje, jak może banować jak tutaj:
if (in_array($userip,$banip))
sprawdza się tablice a nie plik...?

Zacznij używać manuala bo ta rozmowa do niczego nie prowadzi. Zobacz co zwraca file()" title="Zobacz w manualu php" target="_manual

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ? Poczytaj sobie o funkcji file()

No ok, sory, ale zrobiłam wszystko tak jak napisaliście i nic... ani komunikatu o błędzie ani bana.

Problem leży w tzw. whitespaces.
Normalnie należy przeiterować całą tablicę i usunąć białe znaki za pomocą funkcji rtrim(). Jednak można tez "na lewo" - dodać znak końca lini.

[PHP] pobierz, plaintext 
<?php
if (in_array($userip."n", $banip)) {
?>
[PHP] pobierz, plaintext 

Pozdrawiam.

Wielkie dzięki, działa (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Teraz jeszcze chciałabym dodać do tego to:

[PHP] pobierz, plaintext 
<?php
$replace = array( '*' => '([0-9]{1,3})', '?' => '([0-9]{1})', '.' => '.' );
?>
[PHP] pobierz, plaintext 

Tylko za bardzo nie wiem gdzie to wstawić i czy ta zmienna jest dobrze zadeklarowana w mojej sytuacji?

Człowieku zastosuj lepiej jakieś gotowe rozwiązanie Temat: Banowanie z maskami i