[php] skrypt banowania, problem z kilkoma IP Opcje

[pitu]

Witam

Mam skrypt ban listy na swojej stronie, korzysta ona z pliku ban.txt niby wszystko ok jak mam dopisane 1 ip do działa ale jak dopiszę następne np. po spacji albo enterze to już nie działa(osoby z tymi ip mogą wejść na stronę) czy ktoś może zobaczyć ten skrypt i w razie czego przerobić go aby można było dopisywać ip po spacji czyli np. 53.21.252.253 53.21.252.257

Oto ten skrypt:

-kod wklejany do index.php

[PHP] pobierz, plaintext 
<?php
include( 'cfg/ban.lib.php' );
   if( ban( file( 'cfg/ban.txt' ), $_SERVER['REMOTE_ADDR'] ) ) {
	  die( 'Twoje ip jest zbanowane'.$_SERVER['REMOTE_ADDR'].'' );
   }
?>
[PHP] pobierz, plaintext 

-plik ban.lib.php

[PHP] pobierz, plaintext 
<?php
function ban( $arrList, $ip ) {
	  if( in_array( $ip, $arrList ) ) {
		 return TRUE;
	  }
	  $replace = array( '*' => '([0-9]{1,3})', '?' => '([0-9]{1})', '.' => '.');
 
	  foreach( $arrList AS $key => $value ) {
		  if( preg_match( '#^'.strtr( $value, $replace ).'$#', $ip ) ) {
			 return TRUE;
		  }
 
	  }
 
	  return FALSE;
   }
?>
[PHP] pobierz, plaintext 

Z góry dziękuję za pomoc

Ten post edytował pitu120 10.04.2008, 22:37:55

[Lonas]

Spróbuj tak

[PHP] pobierz, plaintext 
<?php
include( 'cfg/ban.lib.php' );
   if( ban( file_get_contents( 'cfg/ban.txt' ), $_SERVER['REMOTE_ADDR'] ) ) {
	  die( 'Twoje ip jest zbanowane'.$_SERVER['REMOTE_ADDR'].'' );
   }
?>
 
//drugi plik
 
<?php
function ban( $arrList, $ip ) 
{
	 $arrIP = explode(" ",$arrList); 
 
	 if( in_array( $ip, $arrIP ) ) 
		 return TRUE;
	 else
		 return FALSE;
  }
?>
[PHP] pobierz, plaintext 

Ten post edytował Lonas 11.04.2008, 06:44:15

[nevt]

funkcja in_array() nie znajduje ci IP ponieważ standardowo file() w każdym wierszu zwraca też znaki końca linii, zmień na:

[PHP] pobierz, plaintext 
<?php
if( ban( file( 'cfg/ban.txt', FILE_IGNORE_NEW_LINES), $_SERVER['REMOTE_ADDR'] ) ) {
?>
[PHP] pobierz, plaintext 

i będzie działać. kolejne wpisy IP umieszczaj w osobnych liniach pliku, bez żadnych spacji przed lub po... powodzenia.

[pitu]

To tak kod podany przez @Lonas troszkę mniej mniej interesuje, ponieważ chciałbym mieć banowanie z maskami (np. 54.65.*) a po zmianie podanej przez @nevt jest tak, że wstawiam np. tak ip do txt:

[HTML] pobierz, plaintext 
83.21.253.52
83.21.253.54
83.21.253.51
[HTML] pobierz, plaintext 

To wtedy banuje tylko podany jako ostatni ip, gdy wstawię swój ip jako np. pierwszy albo drugi od końca to normalnie wchodzę na stronę.

[nevt]

czy ten plik tworzysz pod windozą jeżeli tak, to wróć do pierwotnej wersji (z pierwszego posta), a pierwszej linijce funkcji ban() dopisz:

[PHP] pobierz, plaintext 
<?php
function ban( $arrList, $ip ) {
  $arrList = str_replace(array("r", "\n"), array('', '') $arrList);
  if( in_array( $ip, $arrList ) ) {
...
?>
[PHP] pobierz, plaintext 

[pitu]

Kurcze zmieniłem tak jak podał kolega @nevt, przynajmniej z tego co ja zrozumiałem miałem zmienić w pliku ban.lib.php i zmieniłem tak:

[PHP] pobierz, plaintext 
<?php
function ban( $arrList, $ip ) {
$arrList = str_replace(array("r", "\n"), array('', '') $arrList);
	  if( in_array( $ip, $arrList ) ) {
		 return TRUE;
	  }
	  $replace = array( '*' => '([0-9]{1,3})', '?' => '([0-9]{1})', '.' => '.');
 
	  foreach( $arrList AS $key => $value ) {
		  if( preg_match( '#^'.strtr( $value, $replace ).'$#', $ip ) ) {
			 return TRUE;
		  }
 
	  }
 
	  return FALSE;
   }
?>
[PHP] pobierz, plaintext 

Jeśli to miało być zrobione w ten sposób to lipa, bo wywala błąd i strona się nie wyświetla

[Crozin]

[PHP] pobierz, plaintext 
<?php
 
include( 'cfg/ban.lib.php' );
if(ban('cfg/ban.txt', $_SERVER['REMOTE_ADDR'])){ 
  die( 'Twoje ip jest zbanowane: '.$_SERVER['REMOTE_ADDR'].'</b>' ); 
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
function ban($filename, $remoteAddr){
  $fileContents = file($filename, FILE_IGNORE_NEW_LINES);
  $replace = array( '*' => '([0-9]{1,3})', '?' => '([0-9]{1})');  
 
  foreach($fileContents as $line){ 
	if(preg_match( '#^' . strtr($value, $replace) . '$#', $remoteAddr)) 
	  return true; 
 
  return false;
}
 
?>
[PHP] pobierz, plaintext 

[pitu]

W tym kodzie co podałeś brakowało { za

[PHP] pobierz, plaintext 
<?php
if(preg_match( '#^' . strtr($value, $replace) . '$#', $remoteAddr))
?>
[PHP] pobierz, plaintext 

i

}
}
za

[PHP] pobierz, plaintext 
<?php
return true;
?>
[PHP] pobierz, plaintext 

Bo jak ich nie było wywalało bład ale mimo, że poprawiłem dodając te znaki niby juz nie było blędów na stronie ale mimo dopisywania do txt ip nie banowało

[nevt]

zabrakło jednego przecinka!!! spróbuj raz jeszcze:

[PHP] pobierz, plaintext 
<?php
$arrList = str_replace(array("r", "\n"), array('', ''), $arrList);
?>
[PHP] pobierz, plaintext 

[pitu]

Dzięki @nevt teraz jest ok