User bestätigt mehr Mails als verschickt wurden!?

[Siggi84]

MFG Lokutos

Ja, nur wie macht er das.

Für jedes Problemchen/Fehler/Bug/etc... muss es ja eine Lösung geben.

[hankfromhelvete]

Würdet ihr an meiner Stelle den User sperren?

Würde ich nicht machen, solange du nicht weißt ob es ein Fehler oder Absicht des Users ist.

[jpwfour]

Im Prinzip gibt es ja eine gute Prüfung bei Mailbestätigung, in der ich so erstmal keine groben Fehler erkennen kann. Und wenn die Einträge nicht doppelt vorhanden sind, bleibt noch das mit der race condition, evtl. benutzt er ein Klickprogramm, welches "gleichzeitiges" Aufrufen natürlich besser hinbekommt als ein Mensch.
Wobei er es dann schon absichtlich darauf anlegen müsste, die meisten Fakeskripte geben sich ja mit einer 1x Bestätigung schon zufrieden

Sehe 2 Möglichkeiten:
- User sperren, da selbst wenn das keine Absicht ist, er sich dessen dennoch bewusst ist (mehr Vergütung, +1 Punkte mehr in Rallye) und das nicht meldet beim Admin, schon ein Grund (ausserdem sind die Ausreden doch immer spannend die dann kommen )
- pcheck.php noch besser absichern:

Optimal wären Tabellen-/Zeilensperren, aber etwas kompliziert einzubauen, evtl reicht es auch, die Zeile:

db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=1 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid']." LIMIT 1"); 

die normalerweise erst nach den ganzen Rallyes Bilanzen etc kommt, direkt nach:

if (($mail['start']+$mail['aufendhalt']-1) <= time() AND $mail['status'] == 0) { 

zu verschieben und noch zu erweitern:

db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=1 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid']." AND status=0 LIMIT 1"); 

danach kann man dann mit:

if(mysql_affected_rows()==1){
//rallyes etc.
}else{
$headmsg = 'Doppelt bestätigt! Paidmail ungültig!';
      db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=2 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid']." LIMIT 1");
} 

Das doch schon ganz gut verhindern.

Sieht dann etwa so aus der Abschnitt:

if (($mail['start']+$mail['aufendhalt']-1) <= time() AND $mail['status'] == 0) {
  db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=1 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid']." AND status=0 LIMIT 1");
  if(mysql_affected_rows()==1){
     //rallyes etc. bis:
    $headmsg = $mail['verdienst'].' '.$waehrung.' gutgeschrieben!';
  }else{
  $headmsg = 'Doppelt bestätigt! Paidmail ungültig!';
      db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=2 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid']." LIMIT 1");
  }
} else {
      $headmsg = 'Wartezeit umgangen! Paidmail ungültig!';
      db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=2 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid']." LIMIT 1");
    } 

[Siggi84]

So, jetzt kann ich euch mal wieder updaten

Hatte leider bissel streß, musste mit Hundi in die Tierklinik.

Seitdem ich das "Limit 1" herausgenommen habe, ist es nicht wieder passiert.

Läuft jetzt seit 3 Tagen ohne Abweichung!

Nur wie kann das jetzt gehen, wenn doch keine doppelten Einträge drin waren, oder gerade nur Zufall?

Halte euch auf dem laufenden!

Danke nochmal an ALLE!!!

[Hardy]

Verstehe ich jetzt gerade nicht

Was soll es denn mit den Limit zu tun haben?
Vermutlich hast du noch was anderes verändert was ausschlaggebend war oder doch nur zufall (was ich nicht glaube)

[Siggi84]

Verstehe ich jetzt gerade nicht

Was soll es denn mit den Limit zu tun haben?
Vermutlich hast du noch was anderes verändert was ausschlaggebend war oder doch nur zufall (was ich nicht glaube)

Nein was anderes habe ich nicht gemacht!

Nur aus...

             db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=1 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid']." LIMIT 1"); 

... das

             db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=1 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid'].""); 

So wie es jpwfour Hier beschrieben hat!

[jpwfour]

Wobei, hab mir grad mal die Tabelle vms_paidmails_empfaenger angeschaut, an sich hat die nen Unique Index auf uid, tan, insofern dürfte das Limit 1 irrelevant sein (da keine doppelten Einträge aus dieser Kombination auftreten können).

Evtl. liest der User ja auch jetzt hier mit und unterlässt es daher