WBCE CMS Forum

WBCE CMS – Way Better Content Editing.

Du bist nicht angemeldet.

Ankündigung

WBCE CMS 1.6.9 verfügbar / available - wichtiges Sicherheitsudate / important security update

#1 23.05.2021 14:00:14

giz
Mitglied

[gelöst] fehlermeldungen reviews+ratings nach php8 Update

auch Reviews (1.4.4) schmeißt ein paar Meldungen aus:

2021-05-23T12:47:02+00:00 [Warning] /modules/reviews/reviews.class.php:[570] from /modules/reviews/view.php:[35] Reviews->like "Undefined array key "HTTP_CLIENT_IP""
2021-05-23T12:47:02+00:00 [Warning] /modules/reviews/reviews.class.php:[571] from /modules/reviews/view.php:[35] Reviews->like "Undefined array key "HTTP_X_FORWARDED_FOR""
2021-05-23T12:47:02+00:00 [Warning] /modules/reviews/reviews.class.php:[570] from /modules/reviews/view.php:[35] Reviews->like "Undefined array key "HTTP_CLIENT_IP""
2021-05-23T12:47:02+00:00 [Warning] /modules/reviews/reviews.class.php:[571] from /modules/reviews/view.php:[35] Reviews->like "Undefined array key "HTTP_X_FORWARDED_FOR"" 

cool

Beitrag geändert von florian (23.05.2021 17:36:26)


WBCE 1.6.9 - PHP 8.4.27 - All-inkl
Ein Leben ohne Hund ist möglich, aber sinnlos
#Akkusativ ist kein Verbrechen

Online

#2 23.05.2021 14:19:05

florian
Administrator

Re: [gelöst] fehlermeldungen reviews+ratings nach php8 Update

liegt an reviews.class.php, Zeile 570f.:

$client  = @$_SERVER['HTTP_CLIENT_IP'];
$forward = @$_SERVER['HTTP_X_FORWARDED_FOR'];

- die Konstruktion, mit @ Fehler zu unterdrücken, wurde in PHP 8 abgeschafft.
Die Zeilen müssen also geändert werden - ungefähr so (nicht getestet)

$client = '';
$forward ='';
if (isset($_SERVER['HTTP_CLIENT_IP'])) {
  $client = $_SERVER['HTTP_CLIENT_IP'];
}
if (isset($_SERVER['HTTP_X_FORWARDED_FOR'])) {
  $forward = $_SERVER['HTTP_X_FORWARDED_FOR'];
}

Offline

#3 23.05.2021 15:18:05

giz
Mitglied

Re: [gelöst] fehlermeldungen reviews+ratings nach php8 Update

Eingebaut, Meldung noch identisch vorhanden.  wink danke für Hirnschmalzerwärmung am Sonntag!

Update:
grrrrrrrrrrrrrr kack browsercache... mad  mad  mad  mad
lol
Meldungen sind weg, danke Flash-Florian

Beitrag geändert von giz (23.05.2021 15:22:48)


WBCE 1.6.9 - PHP 8.4.27 - All-inkl
Ein Leben ohne Hund ist möglich, aber sinnlos
#Akkusativ ist kein Verbrechen

Online

#4 23.05.2021 15:39:31

florian
Administrator

Re: [gelöst] fehlermeldungen reviews+ratings nach php8 Update

Hm, nein, das kann eigentlich nicht am Browsercache liegen.
Ich habe es jetzt aber mal selbst getestet. Versuch mal so:

 if (array_key_exists('HTTP_CLIENT_IP',$_SERVER)) {
			$client  = $_SERVER['HTTP_CLIENT_IP'];
		} else {
			$client ='';
		}
		if (array_key_exists('HTTP_X_FORWARDED_FOR',$_SERVER)) {
			$forward = $_SERVER['HTTP_X_FORWARDED_FOR'];
		} else {
			$forward ='';
		}

Außerdem noch die Zeile

if(time() - $last < 60 * $minutes) return time() - $last; //return true;

ändern zu

if(time() - intval($last) < 60 * $minutes) return time() - intval($last); //return true;

Offline

#5 23.05.2021 16:37:04

giz
Mitglied

Re: [gelöst] fehlermeldungen reviews+ratings nach php8 Update

habe jetzt mal testweise div kommentare abgegeben, gelöscht, bewertet etc.
keine errorlog-meldungen so weit. thumb_up


WBCE 1.6.9 - PHP 8.4.27 - All-inkl
Ein Leben ohne Hund ist möglich, aber sinnlos
#Akkusativ ist kein Verbrechen

Online

Fußzeile des Forums

up