Compare commits
No commits in common. "645e98cd6a17259dfb9f8429a07352b7dc537b6f" and "08394be35ec1c63803652c5a1bb4ab881c293660" have entirely different histories.
645e98cd6a
...
08394be35e
|
@ -92,11 +92,6 @@ class RateLimiter {
|
|||
}
|
||||
|
||||
public function attempt($username, $ipAddress) {
|
||||
// Skip rate limiting for whitelisted IPs
|
||||
if ($this->isIpWhitelisted($ipAddress)) {
|
||||
return true;
|
||||
}
|
||||
|
||||
// Clean old attempts
|
||||
$this->clearOldAttempts();
|
||||
|
||||
|
@ -113,10 +108,10 @@ class RateLimiter {
|
|||
}
|
||||
|
||||
public function tooManyAttempts($username, $ipAddress) {
|
||||
$sql = "SELECT COUNT(*) as attempts
|
||||
FROM {$this->tableName}
|
||||
WHERE ip_address = :ip
|
||||
AND username = :username
|
||||
$sql = "SELECT COUNT(*) as attempts
|
||||
FROM {$this->tableName}
|
||||
WHERE ip_address = :ip
|
||||
AND username = :username
|
||||
AND attempted_at > datetime('now', '-' || :minutes || ' minutes')";
|
||||
|
||||
$stmt = $this->db->prepare($sql);
|
||||
|
@ -131,7 +126,7 @@ class RateLimiter {
|
|||
}
|
||||
|
||||
public function clearOldAttempts() {
|
||||
$sql = "DELETE FROM {$this->tableName}
|
||||
$sql = "DELETE FROM {$this->tableName}
|
||||
WHERE attempted_at < datetime('now', '-' || :minutes || ' minutes')";
|
||||
|
||||
$stmt = $this->db->prepare($sql);
|
||||
|
@ -141,10 +136,10 @@ class RateLimiter {
|
|||
}
|
||||
|
||||
public function getRemainingAttempts($username, $ipAddress) {
|
||||
$sql = "SELECT COUNT(*) as attempts
|
||||
FROM {$this->tableName}
|
||||
WHERE ip_address = :ip
|
||||
AND username = :username
|
||||
$sql = "SELECT COUNT(*) as attempts
|
||||
FROM {$this->tableName}
|
||||
WHERE ip_address = :ip
|
||||
AND username = :username
|
||||
AND attempted_at > datetime('now', '-' || :minutes || ' minutes')";
|
||||
|
||||
$stmt = $this->db->prepare($sql);
|
||||
|
|
Loading…
Reference in New Issue