User bestätigt mehr Mails als verschickt wurden!?

[Siggi84]

Guten Morgen,

trotz das in der Datenbank kein doppelter Eintrag war, wieder das gleiche!

Log von gestern:

01.01. 19:39:14 - 2aAQRy9RnaqSMK - 7000.00
01.01. 19:39:14 - 2aAQRy9RnaqSMK - 7000.00
01.01. 19:39:16 - xcbnr7kShM93tf - 7000.00
01.01. 19:39:17 - B9bHgP8xhvZsvM - 7000.00
01.01. 19:39:18 - nKY3KVKs4ses0T - 7000.00
01.01. 19:39:21 - N0bkQOM7JPzou0 - 7000.00
01.01. 19:39:23 - GZPIyiL5xp61JB - 7000.00
01.01. 19:39:25 - DLiTmqe7DbO0Kr - 7000.00

Wieder das gleiche, nur bei dem einen User!

[didith1207]

ich wurde von einem sponsor auf das selbe problem hingewiesen ich hatte das aber auf dem nonlose vms und meine mails verschickt... keine doppelten einträge in der DB und trotzdem hat ein User ständig eine davon 2x bestätigt

[Siggi84]

Und wieder...

Log von heute:

02.01. 12:05:01 - rOKNNI8cOaseWi - 6500.00
02.01. 12:10:39 - sdFcFDQM5asVUk - 6500.00
02.01. 12:33:06 - xhYTpF7PYthQRA - 6500.00
02.01. 16:01:38 - BXetIsYyIw2jAp - 6500.00
02.01. 16:01:40 - PW700O6ELR4rCP - 6500.00
02.01. 16:01:42 - JetvPd0PnFyJAA - 6500.0
02.01. 16:01:46 - nvZNGPaP1AYGTk - 6500.00
02.01. 16:01:46 - 6s0IhCULKFl6rX - 6500.00
02.01. 16:01:46 - 6s0IhCULKFl6rX - 6500.00
02.01. 16:01:47 - sbWPEYrbABr4KQ - 6500.00
02.01. 16:01:49 - SGuMAvxI26Kunl - 6500.00

Trotz das kein Eintrag doppelt ist, habe ich jetzt den Tipp von jpwfour mal gemacht...

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

das Limit 1 weglässt, also:

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

Dann würde bei doppelten TANs jede auf "bestätigt" gesetzt werden (nur für die jewelige UID natürlich), einen Versuch ist es wert (pcheck.php)

... mal sehen was morgen ist.

Würdet ihr an meiner Stelle den User sperren?

Ist zwar nicht meine Art, aber wohl die letzte Maßnahme!?

[Lokutos]

Und wieder...

Log von heute:



02.01. 16:01:46 - 6s0IhCULKFl6rX - 6500.00
02.01. 16:01:46 - 6s0IhCULKFl6rX - 6500.00

MFG Lokutos

[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!