✔ IP Sperre mit Refferer erweitern

[PC Freak]

Hallo, ich habe damals zusammen mit einem anderen User eine IP Sperre programmiert. Klappt auch super, doch möchte ich nun eine kleine Änderung vornehmen und krieg es ohne PHP Kenntnisse nicht hin:

Unzwar habe ich diese Zeile hinzugefügt:

$_SERVER["HTTP_REFERER"]='google' // Und wenn der Referer Google ist

Wie muss diese Zeile richtig lauten ? (anführungszeichen usw.)

Also wenn die IP des Benutzers noch nicht in der Datenbank auftaucht und der Referer Google ist, soll etwas ausgegeben werden:

$pruef=mysql_query("SELECT ip FROM ipsperre WHERE ip='$ip'") or die(mysql_error());
if(@!mysql_fetch_array($pruef)) { // Wenn die IP nicht gefunden wurde
$_SERVER["HTTP_REFERER"]='google' // Und wenn der Referer Google ist

    mysql_query("INSERT INTO ipsperre (ip, time) VALUES ('$ip', '$jetzt')") or die(mysql_error());
    echo $zugriff;

    // IP wurde gefunden
} else { // ansonsten Verweigerung
    echo "kein Zugriff";
}
?>

 

[merzi86]

Wenn ich dich richtig verstanden habe soll das Skript erweitern, dass noch überprüft wird, ob der Refferer Google ist.
Dann kann es so aussehen:

$pruef=mysql_query("SELECT ip FROM ipsperre WHERE ip='$ip'") or die(mysql_error());
if(@!mysql_fetch_array($pruef) && $_SERVER["HTTP_REFERER"]=='google' ) {

    mysql_query("INSERT INTO ipsperre (ip, time) VALUES ('$ip', '$jetzt')") or die(mysql_error());
    echo $zugriff;

    // IP wurde gefunden
} else { // ansonsten Verweigerung
    echo "kein Zugriff";
}
?>

 

[hmmNaGut]

Soll google in die Datenbank gespeichert werden

Was das stückencode für mich sagt:

f(@!mysql_fetch_array($pruef)) { // Wenn die IP nicht gefunden wurde

$_SERVER["HTTP_REFERER"]='google';
 // Ja IP nicht gefunden setzen wir REFERER auf google.
// Weil es so viel sinn macht speichern wir anschliessend in 
// die Datenbank

mysql_query("INSERT INTO ipsperre (ip, time) VALUES ('$ip', '$jetzt')") or die(mysql_error());
echo $zugriff;

Ich vermute du willst wenn es google ist
das es die Insert abfrage überspringt

$pruef=mysql_query("SELECT ip FROM ipsperre WHERE ip='$ip'") or die(mysql_error());
if(@!mysql_fetch_array($pruef) && $_SERVER["HTTP_REFERER"]!='google'  ) { // Wenn die IP nicht gefunden wurde


    mysql_query("INSERT INTO ipsperre (ip, time) VALUES ('$ip', '$jetzt')") or die(mysql_error());
    echo $zugriff;

    // IP wurde gefunden
} else { // ansonsten Verweigerung
    echo "kein Zugriff";
}
?>

Ohne Gewähr

 

[Flex]

Der Referer wird aber nicht nur "google" beinhalten, deshalb empfehle ich eine Prüfung mit [phpf]strpos[/phpf].

 

[hmmNaGut]

Kann sein das mein Vorposter recht hat.
mit der Aussagen logik hab ich es ein bisschen

Ich probier halt gerne

if(@!mysql_fetch_array($pruef) && $_SERVER["HTTP_REFERER"]!='google'  )

oder
if(@!mysql_fetch_array($pruef) && $_SERVER["HTTP_REFERER"]=='google'  )

 

[Flex]

Ich meinte eher:

if(@!mysql_fetch_array($pruef) && strpos($_SERVER["HTTP_REFERER"], 'google' ) === false )

Das würde jetzt prüfen:
IP ist nicht in der Datenbank UND der Referer ist nicht Google.

Grundsätzlich solltest du aber bedenken, dass der Referer nicht immer gesetzt ist.

 

[hmmNaGut]

Felix Jacobi hat natürlich Recht zu dem gibt es ja auch noch
msn und und und...

 

[PC Freak]

if(@!mysql_fetch_array($pruef) && strpos($_SERVER["HTTP_REFERER"], 'google' ) === false )

Das würde jetzt prüfen:
IP ist nicht in der Datenbank UND der Referer ist nicht Google.

Die Ip darf noch nicht in der Datenbank sein und der Referer muss Google sein.

Mit dem Code klappt das leider irgendwie nicht. Scheint so als erkenne dass Script nicht, ob ich von Google komme:

$sperre=time()-3600;
$jetzt=time();
$ip = getip();

mysql_query("DELETE FROM ipsperre WHERE time<$sperre") or die(mysql_error()); // Löscht Einträge, die veraltet sind

$pruef=mysql_query("SELECT ip FROM ipsperre WHERE ip='$ip'") or die(mysql_error());
if(@!mysql_fetch_array($pruef) && $_SERVER["HTTP_REFERER"]=='google' ) {

    mysql_query("INSERT INTO ipsperre (ip, time) VALUES ('$ip', '$jetzt')") or die(mysql_error());
    echo $werbung;

    // IP wurde gefunden
} else { // ansonsten Verweigerung
    echo "";
}
?>

 

[hmmNaGut]

Wenn es google sein muss wenn es die IP nicht gibt
dann statt false true

VOM Felix
if(@!mysql_fetch_array($pruef) && strpos($_SERVER["HTTP_REFERER"], 'google' ) ===true )

 

[Flex]

Danke für die Verbesserung, allerdings gibt [phpf]strpos[/phpf] kein true zurück.

if(@!mysql_fetch_array($pruef) && strpos($_SERVER["HTTP_REFERER"], 'google' ) !== false )

hmmNaGut, verwende doch bitte die PHP Tags um Code in die richtige Box zu tun und das Highlight zu aktivieren.

[php ][/php ]

Ohne Leerzeichen, versteht sich.