Ip ban class met text file
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
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.
Om te reageren heb je een account nodig en je moet ingelogd zijn.
- Details
Door:
Pepijn- 3 jaar geleden
- 1.186 x bekeken
- PHP scripts opties
- PHP algemeen
- Nieuwste PHP scripts
- PHP script toevoegen
PHP hulp
0 seconden vanaf nu