Ip ban class met text file
Door Pepijn , 18 jaar geleden, 5.926x bekeken
Beste medephpers,
Ik had laatst een script nodig om iemand een x aantal uur te verbannen van een pagina. Op mijn login pagina moest een bezoeker 24 uur verbannen worden als deze het wachtwoord meer dan 5 x fout had.
Vandaar dat ik deze class geschreven heb. Ik ben pas een week bezig met programmeren in classes dus alle kritiek is welkom.
Mvg
Pepijn
Gesponsorde koppelingen
PHP script bestanden
Er zijn 6 reacties op 'Ip ban class met text file'
0 
-0 
-0
Nog even aan want meer hergebruik denken:
Code (php)
1
2
3
4
5
6
7
2
3
4
5
6
7
<?php
public $filename = NULL;
function __construct(filename) {
$this->$filename = $filename;
}
?>
public $filename = NULL;
function __construct(filename) {
$this->$filename = $filename;
}
?>
0 -0
-0
Waarom sla je alles op in $this? $content gebruik je niet buiten de methods, dus dan kan je beter een normale variabele gebruiken. En mist $this->position in is_banned? Ik zie hem nergens terugkomen. Bedoelde je daar niet $this->content?
Je manier van ip-adres ophalen werkt averechts. Headers zijn heel gemakkelijk aan requests toe te voegen. Ik hoef nu alleen maar een header Client-IP of X-Forwarded-For mee te sturen met mijn request, en jij pakt het verkeerde adres. Op die manier zou ik bijvoorbeeld jouw ip-adres kunnen verbannen ipv mijn echte adres.
Misschien is het nog een mooie verbetering als je ban_ip() de mogelijkheid geeft om een ander ip-adres te verbannen. Dan kan je deze class ook gebruiken om bijvoorbeeld mensen uit je gastenboek te verbannen wanneer zij ontoepasselijke berichten hebben achtergelaten.
Je manier van ip-adres ophalen werkt averechts. Headers zijn heel gemakkelijk aan requests toe te voegen. Ik hoef nu alleen maar een header Client-IP of X-Forwarded-For mee te sturen met mijn request, en jij pakt het verkeerde adres. Op die manier zou ik bijvoorbeeld jouw ip-adres kunnen verbannen ipv mijn echte adres.
Misschien is het nog een mooie verbetering als je ban_ip() de mogelijkheid geeft om een ander ip-adres te verbannen. Dan kan je deze class ook gebruiken om bijvoorbeeld mensen uit je gastenboek te verbannen wanneer zij ontoepasselijke berichten hebben achtergelaten.
0 -0
-0
IP bans zijn bagger aangezien er honderd-duizend-miljoen mogelijkheden zijn om die te omzeilen. Je hebt er eigenlijk niks aan.
Ook vind ik het niet goed dat je gewoon een txt bestand gebruikt. Gebruik liever bijvoorbeeld xml in combinatie met dom.
Ook kan je beter PHP_EOL gebruiken i.p.v. \r\n aangezien dat niet op elk platform (windows, linux, mac) een new line is.
Ook vind ik het niet goed dat je gewoon een txt bestand gebruikt. Gebruik liever bijvoorbeeld xml in combinatie met dom.
Ook kan je beter PHP_EOL gebruiken i.p.v. \r\n aangezien dat niet op elk platform (windows, linux, mac) een new line is.
0 -0
-0
\r\n werkt op alle platformen als een newline, PHP_EOL is afhankelijk van het besturingssysteem. Zou je PHP_EOL gebruiken, en daarna het tekstbestand van een linux-server naar een windows-server verhuizen, dan zou het niet meer werken. Wat dat betreft is het beter om wel \r\n te gebruiken, en niet PHP_EOL.
In XML zie ik niet echt een voordeel hier. XML is leuk als transport-formaat zodat verschillende services tegen elkaar kunnen praten. Zo'n database-bestand als dit zou ik liever zo binair mogelijk willen hebben, zodat het snel in te laden en aan te passen is. Misschien is een sqlite database wel een beter idee? Dat is ook maar een bestand, maar wel best snel. En je kan erin zoeken, zodat je niet meer je hele bestand hoeft in te laden.
In XML zie ik niet echt een voordeel hier. XML is leuk als transport-formaat zodat verschillende services tegen elkaar kunnen praten. Zo'n database-bestand als dit zou ik liever zo binair mogelijk willen hebben, zodat het snel in te laden en aan te passen is. Misschien is een sqlite database wel een beter idee? Dat is ook maar een bestand, maar wel best snel. En je kan erin zoeken, zodat je niet meer je hele bestand hoeft in te laden.
0 -0
-0
Bedankt voor jullie reacties!
@Karl,
Hoe kan ik dan iemand verbannen? Als IP niet werkt.
Ik heb geen ervaring met xml, ik zal het eens uitzoeken.
@Jelmer
SQlite zie ik wel zitten, ook dat zal ik eens uitzoeken
@Karl,
Hoe kan ik dan iemand verbannen? Als IP niet werkt.
Ik heb geen ervaring met xml, ik zal het eens uitzoeken.
@Jelmer
SQlite zie ik wel zitten, ook dat zal ik eens uitzoeken
Om te reageren heb je een account nodig en je moet ingelogd zijn.
PHP hulp
0 seconden vanaf nu