Anti Flood, Nie kasuje wpisów po określonym czasie :( Opcje

[grzesiek25]

Mam taki o to skrypt anty "floodowy"

[PHP] pobierz, plaintext 
<?php
if (!isset($_SESSION)) {
session_start();
}
if($_SESSION['last_session_request'] > time() - 3){
$ip = $_SERVER["HTTP_X_FORWARDED_FOR"];
if ($ip == "") $ip = $_SERVER["REMOTE_ADDR"];
$db = mysql_connect ("localhost", "login", "hasło");
mysql_select_db ("baza");
mysql_query("INSERT INTO antyflood (IP, TIME) VALUES ('$ip', NOW())");
$timeout = 30; // minutes
$cutoff = (mktime()-($timeout*60));
$query = "DELETE FROM antyflood WHERE TIME < " . $cutoff;
$resuly = mysql_query($query);
exit;
}
$_SESSION['last_session_request'] = time();
?>
[PHP] pobierz, plaintext 

Problem z nim polega na tym że nie kasuje wpisu automatycznie po tych 30 minutach, jest na to jakaś rada ?
wiele sposobów przetestowałem i bez rezultatu.

[be2k]

taki krotki skrypt i tyle bledow

po pierwsze:

[PHP] pobierz, plaintext 
<?php
if (!isset($_SESSION)) {
session_start();
}
?>
[PHP] pobierz, plaintext 

to jest bez sensu, bo zawsze if bedzie spelniony. Dopoki nie wystartujesz z sesja, nie bedziesz mial zadnych zmiennych sesyjnych.

po drugie:

[PHP] pobierz, plaintext 
<?php
$ip = $_SERVER["HTTP_X_FORWARDED_FOR"];
?>
[PHP] pobierz, plaintext 

pobiera ip uzytkownika jesli ten laczy sie przez proxy, a ze nie zawsze tak jest to powinienes sprawdzac czy jest ta zmienna, a jesli jej nie ma to brac ip z

[PHP] pobierz, plaintext 
<?php
$ip = $_SERVER["REMOTE_ADDR"];
?>
[PHP] pobierz, plaintext 

btw. testowales to u siebie lokalnie i Ci dzialalo pobieranie IP

po trzecie, porownaj sobie jak wyglada zmienna:

[PHP] pobierz, plaintext 
<?php
$cutoff = (mktime()-($timeout*60));
?>
[PHP] pobierz, plaintext 

a co zwraca mysql'owy NOW(), zobaczysz ze 'delikatnie' sie roznia...

[grzesiek25]

Pokombinowałem troche nad tym kodem i zmieniłem funkcje mktime na taką w jakiej zapisuje prawidłowo datę i czas czyli teraz wygląda tak samo jak funkcja NOW()

dodałem taki kod

[PHP] pobierz, plaintext 
<?php
$czas = date("Y-m-d H:i:s");
  $secs = 1800;
  $roznica = ($czas - $secs);
  mysql_query("DELETE FROM antyflood WHERE TIME < $roznica");
?>
[PHP] pobierz, plaintext 

jednak dalej bez efektów...

PS. pobieranie IP działa mi bez zarzutu, dodawanie wpisu do bazy po wystąpieniu floodu również gdy nastąpi ponad 3 zapytania na sekundę, jedyny problem to usuwanie wpisów które są już w bazie od 30 minut

[nithajasz]

... i teraz odejmujesz (2008-05-28 12:14:15 - 1800) widzisz w tym sens?

Zmień to na :

[PHP] pobierz, plaintext 
<?php
$czas = now();
  $secs = 1800;
  $roznica = ($czas - $secs);
  mysql_query("DELETE FROM antyflood WHERE TIME < $roznica");
?>
[PHP] pobierz, plaintext 

i w zależności jak masz zapisaną w bazie pozycję antyflood to albo odejmuj od razu lub jeśli masz format datatime to sobie dodaj ta linijkę przed zapytaniem:

[PHP] pobierz, plaintext 
<?php
$cutoff = date("Y-m-d H:i:s", $roznica);
?>
[PHP] pobierz, plaintext 

Ten post edytował nithajasz 28.05.2008, 11:17:59

[grzesiek25]

Nie wiem czy dobrze to poskładałem, wygląda to w ten sposób

[PHP] pobierz, plaintext 
<?
if (!isset($_SESSION)) {
session_start();
}
if($_SESSION['last_session_request'] > time() - 3){
$ip = $_SERVER["HTTP_X_FORWARDED_FOR"];
if ($ip == "") $ip = $_SERVER["REMOTE_ADDR"];
	$db = mysql_connect ("localhost", "login", "hasło");
mysql_select_db ("baza");
mysql_query("INSERT INTO antyflood (IP, TIME) VALUES ('$ip', NOW())");
$czas = now();
$secs = 1800;
$roznica = ($czas - $secs);
$cutoff = date("Y-m-d H:i:s", $roznica);
$query = "DELETE FROM antyflood WHERE TIME < " . $cutoff;
$resuly = mysql_query($query);
exit;
}
$_SESSION['last_session_request'] = time();
?>
[PHP] pobierz, plaintext 

PS. format czasu w bazie to DATETIME

jednak teraz pojawia się błąd

Fatal error: Call to undefined function now()

Ten post edytował grzesiek25 28.05.2008, 11:43:45

[nithajasz]

Zamień funkcję now() na time() i powinno być dobrze..

[grzesiek25]

Zamień funkcję now() na time() i powinno być dobrze..

Zmieniłem, teraz już błędu nie ma ale nadal wpisu nie kasuje mimo że zmieniłem czas dodania wpisu odejmując ponad pół godziny.

[nithajasz]

[PHP] pobierz, plaintext 
<?php
session_start();
 
if($_SESSION['last_session_request'] > time() - 3){
$ip = $_SERVER[&#092;"HTTP_X_FORWARDED_FOR\"];
if ($ip == &#092;"\") $ip = $_SERVER[\"REMOTE_ADDR\"];
	$db = mysql_connect (&#092;"localhost\", \"login\", \"hasło\");
mysql_select_db (&#092;"baza\");
mysql_query(&#092;"INSERT INTO antyflood (IP, TIME) VALUES ('$ip', NOW())\");
$minutes = round(1800/60);
$query = &#092;"DELETE FROM antyflood WHERE TIME < SUBDATE( NOW( ) , INTERVAL \".$minutes.\" MINUTE)\"; 
$resuly = mysql_query($query);
exit;
}
$_SESSION['last_session_request'] = time();
?>
[PHP] pobierz, plaintext 

Powinno działać.. Ale jak nie będzie to pisz bo to przerabiane tak na szybko.

Ogólnie to ja bym zrezygnował z pola datetime i zmienił je na timestamp jakoś łatwiej się operuje na nim.

Ten post edytował nithajasz 28.05.2008, 12:33:15

[be2k]

jak nie wiesz dlaczego cos nie dziala to debuguj, wyswietlaj bledy itp to naprawde pomaga.

prawdopodobnie nie kasuje ci rekordow bo masz bledne zapytanie, dodaj apostrofy:

[PHP] pobierz, plaintext 
<?php
$query = "DELETE FROM antyflood WHERE TIME < '" . $cutoff . "'";
?>
[PHP] pobierz, plaintext 

i wywal to:

[PHP] pobierz, plaintext 
<?php
if (!isset($_SESSION))
{
   session_start();
}
?>
[PHP] pobierz, plaintext 

a zostaw samo:

[PHP] pobierz, plaintext 
<?php
session_start();
?>
[PHP] pobierz, plaintext 

[nithajasz]

[quote name='be2k' date='28.05.2008, 13:30:00 ' post='476300']
jak nie wiesz dlaczego cos nie dziala to debuguj, wyswietlaj bledy itp to naprawde pomaga.

prawdopodobnie nie kasuje ci rekordow bo masz bledne zapytanie, dodaj apostrofy:

[PHP] pobierz, plaintext 
<?php
$query = &#092;"DELETE FROM antyflood WHERE TIME < '\" . $cutoff . \"'\";
?>
[PHP] pobierz, plaintext 

To zapytanie i tak nie pójdzie jak chcesz porównać string do liczby?

[be2k]

To zapytanie i tak nie pójdzie jak chcesz porównać string do liczby?

nie string do liczby tylko string do stringa
przeciez w kodzie ma:

[PHP] pobierz, plaintext 
<?php
$czas = time(); //poprawione z now();
$secs = 1800;
$roznica = ($czas - $secs);
$cutoff = date("Y-m-d H:i:s", $roznica);
?>
[PHP] pobierz, plaintext 

[grzesiek25]

Niestety podane przez was metody nie działają.

[nithajasz]

nie string do liczby tylko string do stringa
przeciez w kodzie ma:

[PHP] pobierz, plaintext 
<?php
$czas = time(); //poprawione z now();
$secs = 1800;
$roznica = ($czas - $secs);
$cutoff = date(&#092;"Y-m-d H:i:s\", $roznica);
?>
[PHP] pobierz, plaintext 

Fakt, coś mi się pomerdało

Ten post edytował nithajasz 28.05.2008, 12:56:08

[be2k]

Niestety podane przez was metody nie działają.

a sprawdzasz to w jakikolwiek sposob? jak wygladaja rekordy w bazie?
jak wyglada zapytanie ktore wykonujesz aby usunac rekordy (echo())?
jestes pewny ze to zapytanie sie poprawnie wykonuje?
moze mysql zwraca jakis blad a ty o tym nie wiesz bo nawet tego nie sprawdzasz?

[grzesiek25]

a sprawdzasz to w jakikolwiek sposob? jak wygladaja rekordy w bazie?
jak wyglada zapytanie ktore wykonujesz aby usunac rekordy (echo())?
jestes pewny ze to zapytanie sie poprawnie wykonuje?
moze mysql zwraca jakis blad a ty o tym nie wiesz bo nawet tego nie sprawdzasz?

Zapytanie dodające wpis do bazy wykonuje się poprawnie
w bazie wygląda to tak
IP xx.xx.xx.xx
TIME 2008-5-28 14:00:27
zapytanie kasujące rekordy to

[PHP] pobierz, plaintext 
<?php
DELETE FROM antyflood WHERE TIME < '" . $cutoff . "'";
?>
[PHP] pobierz, plaintext 

włącze pełny error reporting i może czegoś więcej sie dowiem

[be2k]

[PHP] pobierz, plaintext 
<?php
DELETE FROM antyflood WHERE TIME < '" . $cutoff . "'";
?>
[PHP] pobierz, plaintext 

chodzilo mi o to zebys wyswietlil to zapytanie, czyli echo("DELETE FROM...") zeby zobaczyc jak wyglada z wartoscia zmiennej ktora tam wstawiasz

[grzesiek25]

chodzilo mi o to zebys wyswietlil to zapytanie, czyli echo("DELETE FROM...") zeby zobaczyc jak wyglada z wartoscia zmiennej ktora tam wstawiasz

To zapytanie wyświetla mi error

Parse error: syntax error, unexpected T_ECHO

próbowałem tak

[PHP] pobierz, plaintext 
<?php
$query = echo "DELETE FROM antyflood WHERE TIME < '" . $cutoff . "'";
i
$query = echo ("DELETE FROM antyflood WHERE TIME < '" . $cutoff . "'");
?>
[PHP] pobierz, plaintext 

w obu przypadkach ten sam błąd

[em1X]

Lepiej pokaż jak wygląda struktura Twojej bazy. Bo jeżeli time jest typu varchar no ciekaw jestem jak mają działania matematyczne na niej zadziałać. Poza tym nazwy pól lepiej owijać w cudzysłów, bo słowo time może być słowem kluczowym w starszych wersjach.

[PHP] pobierz, plaintext 
<?php
// po ilu minutach usunąć
$_time = 30;
 
// usuwamy rekordy
mysql_query('delete from `tabela` where `time` + '.($_time * 60).') < '.time()) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

Ten post edytował em1X 28.05.2008, 13:27:00

[grzesiek25]

Lepiej pokaż jak wygląda struktura Twojej bazy. Bo jeżeli time jest typu varchar no ciekaw jestem jak mają działania matematyczne na niej zadziałać. Poza tym nazwy pól lepiej owijać w cudzysłów, bo słowo time może być słowem kluczowym w starszych wersjach.

[PHP] pobierz, plaintext 
<?php
// po ilu minutach usunąć
$_time = 30;
 
// usuwamy rekordy
mysql_query('delete from `tabela` where `time` + '.($_time * 60).') < '.time()) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

tak wygląda tabelka

[PHP] pobierz, plaintext 
<?php
CREATE TABLE `antyflood` (
  `IP` char(32) NOT NULL default '',
  `TIME` datetime NOT NULL default '0000-00-00 00:00:00',
  PRIMARY KEY  (`IP`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8;
?>
[PHP] pobierz, plaintext 

przy twoim kodzie TIME nie powinno mieć przypadkiem ustawionego samego typu time ?

[be2k]

To zapytanie wyświetla mi error

Parse error: syntax error, unexpected T_ECHO

próbowałem tak

[PHP] pobierz, plaintext 
<?php
$query = echo "DELETE FROM antyflood WHERE TIME < '" . $cutoff . "'";
i
$query = echo ("DELETE FROM antyflood WHERE TIME < '" . $cutoff . "'");
?>
[PHP] pobierz, plaintext 

w obu przypadkach ten sam błąd

chlopie! wklej tutaj wynik funkcji echo "DELETE FROM..."
podstawy, podstawy, podstawy...