search

boombuler / piraten_map_server

8 stars 5 forks source link

Mail-Adresse nur auf Substring "pirat" prüfen macht wenig Sinn #4

Open thatgim opened 10 years ago

thatgim commented 10 years ago

Eine Mail-Adresse die den String "pirat" enthält und ist keine wirksame Schutzmaßnahme. Dies ist auch im Wiki dokumentiert: http://wiki.piratenpartei.de/Plakatkarte

Aus meiner Sicht gibt es zwei Lösungsansätze: a) Prüfung ausbauen, da nicht jeder aktive Pirat über so eine Adresse verfügt (und sich nicht nur dafür anlegen will). Bei der Europa+Kommunalwahl in Bielefeld hat das für Verwirrung gesorgt und nur einige wenige haben Plakate eingetragen. Andere haben Listen mit Standorten weiter gegeben.

b) Falls die Prüfung als notwendig erachtet wird sollte nicht nur auf den Substring "pirat" geprüft werden sondern auf die Domains der Landesverbäden und ähnlichem.

Wäre der Quellcode mit dieser Prüfung hier vorhanden, bin ich bereit die Änderung durch zuführen und per Pull-Request anzubieten.

DJaeger commented 10 years ago

Es ist bewusst eine Zwischenlösung zwischen den beiden von dir genannten Ansätzen. So existiert eine Hürde, die bisher auch gereicht hat um Trolle abzuwehren, die aber nicht alle Bundesländer ausschließt, die nicht an jedes Mitglied eine offizielle Mail-Adresse vergeben.

Es wird nur auf "pirat" geprüft, um damit die verbreitete Variante "pirat@eigenedomain.de" mit einzuschließen.

Für alle die, die auch keine Mail-Adresse besitzen, die "pirat" beinhaltet wurde die Einladen-Funktion eingeführt, wobei die Prüfung auf "pirat" enfällt, da von einer Verifizierung des Eingeladenen durch den Einladenden ausgegangen wird.

Der Quellcode dieser Prüfung ist lediglich folgender:

if (!isAdmin() && !$correctInvitation) {
    if (stripos($email, 'pirat') === false) {
        $message['type'] = 'error';
        $message['text'] = 'Ungültige E-Mail Adresse! Bitte benutze eine Piraten-Adresse!';
        return false;
    }
}

P.S.: Wie auch im Wiki steht ist dieses Repo nicht die aktuell genutzte Version.

codedust commented 8 years ago

P.S.: Wie auch im Wiki steht ist dieses Repo nicht die aktuell genutzte Version.

Besteht denn die Möglichkeit, irgendwie an die aktuelle Version ranzukommen?

DJaeger commented 8 years ago

Ja, ich werde die aktuelle Version online stellen, sobald ich etwas Zeit dafür finde