[FINAL] [CDB][3.3] Spamsecure

[T-Rex]

Das wäre bei mir für einen Text in kyrillisch ja dann auch ok, aber ein rein deutscher Text?

Was mich am meisten irritiert, dass die Anzahl der ungültigen Zeichen bei mir um 1 zu hoch ist.

Es ist dabei aber auch egal, ob ich firefox oder chromium benutze.

Auf meinem Rechner selber läuft ein openSUSE Leap 15.3 und die Änderung habe ich mit einem Editor und auch noch mal per vi gemacht.

[LukeWCS]

Was mich am meisten irritiert, dass die Anzahl der ungültigen Zeichen bei mir um 1 zu hoch ist.

Wie gesagt, ich kann es nicht nachvollziehen, aber genau das ominöse 34te Byte dürfte das Problem bei dir sein.

Jut, ich schlage folgendes vor, weil wir sonst noch ewig rätseln werden:

1. Nimm Spamsecure von allen Gruppen (setze "Ja") und setze nur deinen Test Benutzer auf "Nie". Dann können wir erstens testen ohne den Betrieb zu stören und zweitens funktioniert Spamsecure in dem Zustand für dich ja eh nicht.

2. Dann baust du meinen Fix wieder ein.

3. Dann fügst du meinen Debug Code direkt unter der besagten foreach Schleife (mit meinem Fix). Sollte dann alles so aussehen:

Code: Alles auswählen

			foreach ($checkarray as $checkchar)
			{
				if (strpos($message, $checkchar, 0) !== false)
				{
					$counter++;
					$matches[] = $checkchar;
				}
			}
			foreach ($matches as $match)
			{
				$bytes = '';
				for ($i = 0; $i < strlen($match); $i++)
				{
					$bytes .= ord(substr($match, $i, 1)) . ', ';
				}
				$bytes = trim($bytes, ', ');
				echo $match . ' = ' . $bytes . '<br>';
			}

4. Dann bitte wieder mit dem Test String testen und hier das Ergebnis des Debugs posten.

а,б,в,г,д,е,ё,ж,з,и,й,к,л,м,н,о,п,р,с,т,у,ф,х,ц,ч,ш,щ,ъ,ы,ь,э,ю,я

[T-Rex]

Hier kommt das Debug-Ergebnis:

Code: Alles auswählen

а = 208, 176
б = 208, 177
в = 208, 178
г = 208, 179
д = 208, 180
е = 208, 181
ё = 209, 145
ж = 208, 182
з = 208, 183
и = 208, 184
й = 208, 185
к = 208, 186
л = 208, 187
м = 208, 188
н = 208, 189
о = 208, 190
п = 208, 191
р = 209, 128
с = 209, 129
т = 209, 130
у = 209, 131
ф = 209, 132
х = 209, 133
ц = 209, 134
ч = 209, 135
ш = 209, 136
щ = 209, 137
ъ = 209, 138
ы = 209, 139
ь = 209, 140
э = 209, 141
ю = 209, 142
я = 209, 143
=
[phpBB Debug] PHP Warning: in file [ROOT]/includes/functions.php on line 4148: Cannot modify header information - headers already sent by (output started at [ROOT]/ext/bruno/spamsecure/event/listener.php:130)
[phpBB Debug] PHP Warning: in file [ROOT]/includes/functions.php on line 4148: Cannot modify header information - headers already sent by (output started at [ROOT]/ext/bruno/spamsecure/event/listener.php:130)
[phpBB Debug] PHP Warning: in file [ROOT]/includes/functions.php on line 4148: Cannot modify header information - headers already sent by (output started at [ROOT]/ext/bruno/spamsecure/event/listener.php:130)
[phpBB Debug] PHP Warning: in file [ROOT]/includes/functions.php on line 4148: Cannot modify header information - headers already sent by (output started at [ROOT]/ext/bruno/spamsecure/event/listener.php:130)

[LukeWCS]

Prima, jetzt kommen wir weiter.

Interessant. Das ist der besagte zweite Fehler den ich erwähnt hatte, der aber bei mir keine funktionelle Auswirkung hat, bei dir aber schon. Gut, dann fixen wir den auch gleich:

Suche

Code: Alles auswählen

$counter = 0;

Füge darüber ein

Code: Alles auswählen

$searchchars = trim($searchchars, ',');

Jetzt teste erneut. Im Idealfall sollte der Test String jetzt 33 Zeichen melden und dein Test Benutzer sollte dann auch in der Lage sein, normal posten zu können.

[T-Rex]

Jetzt zeigt er die richtige Anzahl an nicht erlaubten Zeichen.

Ein normaler Text in lateinischen Buchstaben geht jetzt durch.

Vielen Danke für deine Hilfe, dann kann ich jetzt ja die Gruppenberechtigungen wieder so einstellen wie ich sie vor unserem Test und der Korrektur hatte. Deinen Debug-Code baue ich dann jetzt gleich wieder aus bzw. kommentier ihn erstmal nur aus. Entfernen kann ich ihn dann später immer noch.

[LukeWCS]

@T-Rex

Bitteschön. Ja, Berechtigungen wieder auf den Soll-Zustand bringen. Den Debug Code kannst einfach deaktivieren, falls wir ihn nochmal brauchen sollten, aber ich glaube nicht. Ansonsten hat sich der spätestens beim nächsten Update eh erledigt.

[professor-berni]

Moin,
ich bekomme diese Fehlermeldung, wenn ich einem Benutzer, der sich erst kürzlich registriert hat, per Spamsecure mit "Nie" verbiete, html-Code einzugeben:
[ externes Bild ]
Die Einstellungen dazu für diese Rolle:
[ externes Bild ]
Bug oder Feature?

[chris1278]

Hast du wahrscheinlich den Debug modus im forum aktiviert?

Poste doch mal den inhalt deiner benutzten listener.php die du im ext verzeichnis event findest. Nutze dazu bitte entweder Pastbin oder den bbcode mit dem Button [<>] hier.

[chris1278]

Hier ist die aktuelle version jetzt online zum Download.

Dort sind die Änderungen die Lukewcs geschrieben hat enthalten sowie Die Korrektur einiger Codestellen die durch die Codeprüfer auf phpbb.com angemahnt wurden sind.

Download im Startbeitrag.

[T-Rex]

Hab letztes Wochenende endlich das von mir softwaremäßig betreute Forum auf die aktuelle phpBB Version heben und dabei auch die aktuelle Version von Spamsecure einsetzen können.