agreed im ucp.php?

[Coki]

Moin Moin

Mal eine reine Verständnisfrage, da ich mir zwar den Code genauer angeschaut, aber im Vergleich zu 2er Version noch nicht verstanden habe.

Es geht um folgendes: Bis vor einiger Zeit hatte ich massive Probleme mit per Bot angemeldete Fakeanmeldungen. Durch eine Änderung von agreed in zb. agreeed in der Datei usercp_register.php von phpbb2 konnte ich das Problem völlig entspannt und sehr individuell lösen.

Nun würde mich mal interessieren wie das beim 3er gelöst ist. Die einzige agreed Verknüpfung habe ich beim Button bei der Registrierung gesehen.

Gruss,

Coki

[sepp71]

Ich grabe dieses Thema einfach nochmal aus:
Unter phpbb 2 war ich mit der schlichten Änderung des Variablennamens für agreed gut gefahren (http://www.phpbb.de/viewtopic.php?p=660430#660430).
Jetzt steigen die Spam-Anmeldungen unter phpbb 3.0.4 bei mir gerade wieder - kann man das 1:1 auf 3.0 übertragen? Gibt es irgendwo schon eine aktualisierte Anleitung dafür?
"agreed" kommt im Quelltext jetzt insgesamt recht oft vor, so dass ich Sorge habe, in diesem sensiblen Bereich zuviel oder zuwenig zu ändern.

[root\includes\ucp\ucp_register.php]

Code: Alles auswählen

Line 42 : 		$agreed			= (!empty($_POST['agreed'])) ? 1 : 0;
Line 42 : 		$agreed			= (!empty($_POST['agreed'])) ? 1 : 0;
Line 47 : 		if ($agreed)
Line 67 : 					// Setting back agreed to let the user view the agreement in his/her language
Line 68 : 					$agreed = (empty($_GET['change_lang'])) ? 0 : $agreed;
Line 68 : 					$agreed = (empty($_GET['change_lang'])) ? 0 : $agreed;
Line 87 : 		if (!$agreed || ($coppa === false && $config['coppa_enable']) || ($coppa && !$config['coppa_enable']))
Line 435 : 			'agreed'		=> 'true',

[root\styles\prosilver\template\ucp_agreement.html]

Code: Alles auswählen

Line 23 : 			<input type="submit" name="agreed" id="agreed" value="{L_AGREE}" class="button1" />&nbsp;
Line 23 : 			<input type="submit" name="agreed" id="agreed" value="{L_AGREE}" class="button1" />&nbsp;
Line 24 : 			<input type="submit" name="not_agreed" value="{L_NOT_AGREE}" class="button2" />

[root\ucp.php]

Code: Alles auswählen

Line 62 : 		if ($user->data['is_registered'] || isset($_REQUEST['not_agreed']))

Muss ich da überall alle Vorkommen von 'agreed' durch etwas anderes ersetzen?
Vielen Dank!

Sepp
Edit 05.01.2009: Fehlerhaft mitbenannten zusätzlichen Style entfernt - benutze ausschließlich prosilver

[nickvergessen]

Es dürfte reichen
in den php-Dateien

Code: Alles auswählen

$_POST['agreed']

und im Template

Code: Alles auswählen

name="agreed" id="agreed"

[sepp71]

Super, vielen Dank für die prompte Rückmeldung am Feiertag!

Sepp

[sepp71]

Ich nochmal : Die sich daraus ergebenden zwei Änderungen scheinen leider doch nicht alles zu sein. Die fraglichen Passagen habe ich so insgesamt nur jeweils einmal, nämlich in
ucp_agreement.html

Code: Alles auswählen

<input type="submit" name="agreed" id="agreed" value="{L_AGREE}" class="button1" />&nbsp; -->

und
ucp_register.php

Code: Alles auswählen

$agreed			= (!empty($_POST['agreed'])) ? 1 : 0;

gefunden und einheitlich geändert (3 x agreed in 3 x einverst).
Dann kommt man bei der Registrierung aber immer wieder bei den Benutzungsregelungen aus und nicht wirklich ans Ziel (hatte ursprünlich mal über acht Zeichen, einkürzen hat aber auch nichts gebracht).
Kann es sein, dass die Funktion, die den eigentlichen Vergleich durchführt, irgendwo anders steht?
(Ich benutze ausschließlich prosilver, auch wenn das im letzten Post versehentlich anders aussah)

Weil das im alten phpbb so gut funktioniert hat, wäre ich für einen weiteren Tipp dankbar...
Sepp

P.S.: Template aktualisiert + Cache im ACP gelöscht.

[Dr.Death]

Hallo,

warum verschärfst Du nicht das in phpBB3 vorhandene CAPTCHA ?

[sepp71]

Weil ich rot-grün schwach bin und selbst immer froh bin, wenn ich diese Buchstabenrätsel bis zum dritten Versuch fehlerfrei gelöst bekomme

Stimmt zwar auch, aber die Wahrheit ist natürlich:

Weil ich bis gerade nicht bemerkt hatte, dass das geht und solche erhebliche Effekte hat.
Danke, habe es gerade umgestellt und werde weiter beobachten!

Sepp
(der übrigens am WE wieder mal in Xanten war)

[Wuppi]

Hi

sorry fürs hochholen

Klappt die besagte Modifikation den sauber? Brauch sowas auch bald Im phpBB2 fahr ich mit agreed=zustimmung - keine Bot-Anmeldungen mehr. Gleiches würd ich gerne mit in phpBB3 übernehmen.

Captcha verschärfen kommt nicht in Frage ... das so, gerade für ältere teilweise schon ne zumutung. Ich darf beim phpBB2-Captcha schon regelmässig User manuell anlegen da die das Captcha nicht lesen können

Captcha ist für viele ne Zumutung ... die Variablenänderungen stört keinen Menschen - nur Bots und somit eigentlich die bessere Lösung.

Gruß

[sepp71]

Ich kann Deine Bedenken verstehen, hatte es damals aber mit einem - nach Dr. Death's Tipp nur noch halbherzigen Versuch - leider nicht hinbekommen, wie man liest.

Gruß
Sepp

[Wuppi]

Hi

haste den mal die radikal-Variante probiert: ALLE agreed-Vorkommen in den von dir genannten Dateien umbenannt?

Gruß