Kan dit?

[Coen]

Ik geloof niet dat die var ergens wordt opgeslagen, dus dan hoeft het niet.

Mooi, dan klopt het stukje code zo, helemaal veilig!

[Paul]

Ja natuuurlijk moet dat. Verder, gewoon de code die bas geeft, en daarna dit toevoegen:

Code: Selecteer alles

$this->ip = htmlspecialchars($this->ip);

[Paul]

Ik geloof niet dat die var ergens wordt opgeslagen, dus dan hoeft het niet.

die wordt soms in de db opgeslagen.

[Coen]

Ja natuuurlijk moet dat. Verder, gewoon de code die bas geeft, en daarna dit toevoegen:

Code: Selecteer alles

$this->ip = htmlspecialchars($this->ip);

Dat kan uiteraard ook, het is inderdaad wel slimmer om op het einde alles bij elkaar te pakken. Was te geconcentreerd op de code, om het nut ervan te zien en te doorzien dat het ook op het einde kon!

[emrulez]

Code: Selecteer alles

if ($_SERVER['REMOTE_ADDR'] == '192.168.13.41') {
   if (isset($_SERVER['HTTP_X_FORWARDED_FOR']) && !empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
      $this->ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
   } else {
      $this->ip = $_SERVER['REMOTE_ADDR'];
   }
} else {
   $this->ip = $_SERVER['REMOTE_ADDR'];
   $this->ip = htmlspecialchars($this->ip);
}

IS het zo veilig?

[Coen]

Wel veilig, maar klopt niet helemaal;

Code: Selecteer alles

if ($_SERVER['REMOTE_ADDR'] == '192.168.13.41') {
   if (isset($_SERVER['HTTP_X_FORWARDED_FOR']) && !empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
      $this->ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
   } else {
      $this->ip = $_SERVER['REMOTE_ADDR'];
   }
} else {
   $this->ip = $_SERVER['REMOTE_ADDR'];   
}
$this->ip = htmlspecialchars($this->ip);

Zo zou hij moeten...

[emrulez]

Wel veilig, maar klopt niet helemaal;

Code: Selecteer alles

if ($_SERVER['REMOTE_ADDR'] == '192.168.13.41') {
   if (isset($_SERVER['HTTP_X_FORWARDED_FOR']) && !empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
      $this->ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
   } else {
      $this->ip = $_SERVER['REMOTE_ADDR'];
   }
} else {
   $this->ip = $_SERVER['REMOTE_ADDR'];   
}
$this->ip = htmlspecialchars($this->ip);

Zo zou hij moeten...

DANkje