PHP und IMAP: Mails filtern

  • PHP und IMAP: Mails filtern

    Hi zusammen,

    Folgenden Code habe ich mir zusammengeschustert :) Aber Wenn ich ihn ausführe wird die Variable Message nur beim ersten Durchlauf ausgegeben. Ich glaube das beim zweiten Durchlauf irgendwas mit der Count Variable schiefgeht, kann das sein?

    Quellcode

    1. <?php
    2. $imap = imap_open ("{mail.orczak.de/pop3:110}INBOX", "web2p2", "geändert");
    5. $check = imap_mailboxmsginfo ($imap);
    6. $newmails = $check->Recent;
    9. for($count = 1; $count <= $newmails; $count++)
    10. {
    11. $header = imap_header($imap, $count);
    12. $from = $header->from;
    13. foreach ($from as $id => $object) {
    14. $fromaddress = $object->mailbox . "@" . $object->host;
    15. }
    17. if($fromaddress == "matthias@orczaki.com"){
    19. $info = imap_fetch_overview($imap, $count);
    20. foreach ($info as $msg)
    21. {
    22. }
    23. $message = imap_body($imap, $count);
    24. $text = $msg->to;
    25. $needle = strpos($text, '@');
    26. $newnumber = substr($text, 0,$needle);
    27. echo $newnumber;
    28. echo $message;
    30. $cusid = "3537";
    31. $passwd = "aaaaaa";
    32. $dnis = $newnumber;
    33. $content = $message;
    37. $client = new SoapClient("http://webservices.venali.net/fax/1.0/faxservice.asmx?WSDL",array("trace" => 1, "exceptions" => 0));
    38. $params = array
    39. (
    40. "customerID" => $cusid,
    41. "password" => $passwd,
    42. "phone" => $dnis,
    43. "body" => $content
    44. );
    46. $res = $client->SendLite($params);
    47. print_r($res);
    52. imap_close ($imap);
    53. }
    54. else
    55. {
    56. echo "pech";
    57. }
    59. }
    60. ?>
    Alles anzeigen


    Merci für's reinschaun!

    Matthias
    Das Leben ist binär - du bist eine 1, oder eine 0

  • Wenn du nach Mails von einem bestimmten Absender suchen willst, dann nutze [phpdoc]imap_search[/phpdoc].
    Deine Schleifenlösung ist aber sowieso überflüssig, da es nur einen Absender geben kann ;)

    In #19 bis #21 hast du wieder eine merkwürdige Schleifenlösung. Erst iterierst du durch alles durch und dann benutzt du das letzte Element ($msg->to).

    Stelle erstmal um auf imap_search und baue dann ein paar Ausgaben ein, damit wir wissen wo sich das Script befindet.

    Und bitte diesmal einrücken ;)

  • Kommt das nicht aufs selbe hinaus?

    Zur Zeit holst du alle Message IDs, lädst die Header von allen Message IDs und wertest diesen dann aus indem du den Absender vergleichst.

    Als Ergebnis der IMAP Suche erhältst du alle Message IDs der Nachrichten mit dem bestimmten Absender. Du sparst dir also die Zeilen 10-18 und dazu noch viel Wartezeit die durch die IMAP Anfragen entstehen.

    Was ich in deinem Code vermisse und was die Suche auch beherrscht: Du kannst das Flag UNSEEN hinzufügen. Dann faxt er eine Nachricht nicht jedes mal wieder durch,

  • HI!

    Also, das er es nicht jedesmal faxt wäre natürlich schon praktisch *dummymodeoff* ... ich dachte mir halt das in der Final dann die Nachricht vom Server gelöscht (oder verschoben wird) ....

    Hmmm... ich weiß nicht ob wir das gleiche meinen :)

    In der final wird in einer MySQL eine Row mit Emailadressen sein, ist der Sender einer Email da nicht drinnen wird das Fax nicht versendet. Im ersten Schritt möchte ich es halt mal nur mit einer einzigen und wenn das funzt kommt die Anbindung dran.

    Und nun überlege ich ob IMAP_SEARCH das richtige ist .... meinst du immer noch es wäre besser als mein Weg?

    Matthias
    Das Leben ist binär - du bist eine 1, oder eine 0

  • definitiv.. schau dir den Code mal an...

    Quellcode

    1. <?php
    2. $imap = imap_open("{imap.server.tld}INBOX","user","password");
    3. foreach(imap_search($imap, "FROM \"matthias@orczaki.com\" UNSEEN") as $id)
    4. $header = imap_headerinfo($imap, $id);
    6. echo "Send Fax to ". $header->to ." with contents: ".imap_body($imap, $id);
    7. }
    8. imap_close($imap);
    9. ?>


    etwas kürzer, oder?

  • Hi!

    Wenn ich den Code richtig verstehe passiert folgendes:

    imap_search($imap, "FROM \"matthias@orczaki.com\" UNSEEN"

    sucht in meiner Mailbox nach der obigen Emailadresse, wenn er diese findet wird weitergearbeitet, wenn nicht, dann ned, richtig?

    Okay, nun tritt aber anstelle der einzelnen Emailadresse ein Array mit Emailadressen und darin sind 1000 Emailadressen (als Beispiel) .... und wir gehen weiter davon aus das ich 10 Nachrichten in der Mailbox habe, macht es dann denn auch Sinn deinen Weg zu gehen? Denn dann wird ja 1000x die Mailbox durchsucht, richtig?

    Matthias
    Das Leben ist binär - du bist eine 1, oder eine 0

  • Okay. (Du glaubst nicht wie froh ich bin mal nicht als Trottel dazustehen) :)

    Und wenn man nun genau das erreichen möchte ist mein Script doch okay, oder? Nur das eben bei den wiederholten Schleifendurchläufen die Variablen nicht mehr gefüllt werden, ergo beginnt die Schleife entweder zu spät oder aber mitm Counter passt was nicht, richtig?
    Und da hänge ich momentan ... :(

    Matthias

    P.S. Ich find es immer wieder Klasee euch gefunden zu haben, hier bekommt man wenigstens wirklich Hilfe ...
    Das Leben ist binär - du bist eine 1, oder eine 0

  • Hi!

    Okay, ein bisschen Stolz darf ich sein denke ich - es läuft :)

    Hier der Code den ich mir gebastelt habe:

    Quellcode

    1. <?php
    2. $imap = imap_open ("{mail.hoster.de/pop3:110}INBOX", "web2p2", "changed");
    3. $check = imap_mailboxmsginfo ($imap);
    4. $rescheck = $check->Recent;
    5. $count = 1;
    8. while($count <= $rescheck)
    9. {
    10. $message = imap_body($imap, $count);
    13. $header = imap_header($imap, $count);
    14. $from = $header->from;
    15. $to = $header->to;
    16. foreach ($from as $id => $object) {
    17. $fromaddress = $object->mailbox . "@" . $object->host;
    18. foreach ($to as $id => $object)
    19. $toaddress = $object->mailbox;
    20. }
    22. $verify = "matthias@hosteri.com";
    24. if ( $fromaddress == $verify ) {
    25. echo $count."<br>".$toaddress."<br>".$message."<br>".$fromaddress."<br>";
    26. }
    27. else
    28. {
    29. echo $count." ".$fromaddress." Hier ist der Sender ein anderer"."<br>";
    30. }
    31. $count++;
    33. }
    35. ?>
    Alles anzeigen


    Und nun warte ich auf d0nUt der mir zeigt wie er den Code optimieren könnte (oder andere eben) :) Aber laufen tut er *freu*

    Matze
    Das Leben ist binär - du bist eine 1, oder eine 0

  • ich glaube ich habe jetzt auch durchschaut warum du die beiden foreach schleifen nutzt. 3 Schleifen ineinander sind sehr, sehr böse ;)

    Ich glaube du hast das Array Kapitel übersprungen :P
    An Index und Value kommst du mit [phpdoc]list[/phpdoc] und each

    Quellcode

    1. <?php
    2. $imap = imap_open ("{mail.hoster.de/pop3:110}INBOX", "web2p2", "changed");
    3. $check = imap_mailboxmsginfo ($imap);
    4. $count = 1;
    5. $verify = "matthias@hosteri.com";
    7. while($count <= $check->Recent) {
    8. $message = imap_body($imap, $count);
    9. $header = imap_header($imap, $count);
    11. list($id, $ob) = each($header->from);
    12. $fromaddress = $ob->mailbox . "@" . $ob->host;
    14. $ob = array_pop($header->to);
    15. $toaddress = $ob->mailbox;
    17. if ($fromaddress == $verify) {
    18. echo $count."<br />".$toaddress."<br />".$message."<br>".$fromaddress."<br>";
    19. } else {
    20. echo $count." ".$fromaddress." Hier ist der Sender ein anderer"."<br>";
    21. }
    23. $count++;
    24. }
    25. ?>
    Alles anzeigen


    Kleiner Schönheitsfehler: $object ist ein Schlüsselwort - das solltest du nicht verwenden.

  • *lach*

    Bloody NoOb against Specialist :) Da war der Ausgang ja klar :) ... ich habe ja auch nur geschrieben das der Code läuft, nicht das er valid is *grins* Aber wenigstens geb ich mir Mühe :)

    So, nun muss ich nur noch den "festen" Wert verify durch ein Array ersetzen welches durchsucht wird, und dann ist mein Test beendet (zumindestens der Part) :) Des krieg ich schon irgendwie hin, und dann post ich meinen Code wieder zum verbessern? :)

    MERCI auf jeden Fall,

    Matthias
    Das Leben ist binär - du bist eine 1, oder eine 0

  • So,

    Nachdem ich nicht schlafen konnte hier nun der Code der ein Array prüft und wenn der jeweilige Wert im Array ist wird entsprechend der if Schleife gehandelt.

    Leider ist der Code etwas "langsam" .... also beim Vergleich mit 3 Emailadressen braucht er schon so 1-4 Sekunden, wie des dann bei 1000 Vergleichen wohl aussieht? *grins* ... sicherlich habe ich mal wieder was nicht "optimal" sondern "nur" funktionell gelöst .... vielleicht weiß jemand Rat.

    Hier der Code:

    Quellcode

    1. <?php
    2. $imap = imap_open ("{mail.anbieter.de/pop3:110}INBOX", "web2p2", "changed");
    3. $check = imap_mailboxmsginfo ($imap);
    4. $count = 1;
    5. // $verify = "matthias@hosteri.com";
    7. while($count <= $check->Recent) {
    10. $message = imap_body($imap, $count);
    11. $header = imap_header($imap, $count);
    13. list($id, $ob) = each($header->from);
    14. $fromaddress = $ob->mailbox . "@" . $ob->host;
    16. $mysearch = array("matthias@hoster.com","matthias@nixda.de","matthias@web.de");
    17. if (in_array($fromaddress, $mysearch)) {
    18. $verify = $fromaddress;}
    20. $ob = array_pop($header->to);
    21. $toaddress = $ob->mailbox;
    23. if ($fromaddress == $verify) {
    24. echo $count."<br />".$toaddress."<br />".$message."<br>".$fromaddress.$verify."<br>";
    25. } else {
    26. echo $count." ".$fromaddress." Hier ist der Sender ein anderer".$verify."<br>";
    27. }
    29. $count++;
    30. }
    31. ?>
    Alles anzeigen


    Merci vorab :)

    Matthias
    Das Leben ist binär - du bist eine 1, oder eine 0

  • Da hatte ich mal wieder ne große Klappe glaube ich :)

    Könnte mir jemand die Richtung zeigen wie ich das am besten anstelle?

    In folgende Zeile gehören doch die Werte aus der MySQL, richtig?
    $mysearch = array("matthias@vi.com","matthias@nixda.de","matthias@web.de");


    Die MySQL Abfrage sollte doch so aussehen, oder?

    $lala = mysql_query("SELECT emailaddress FROM allowed");
    $row = mysql_fetch_row($result);

    Und da scheine ich meinen Fehler zu haben, richtig? mysql_fetch_row ist nicht was ich brauche, richtig? Wie also lautet hier der richtige Befehl?

    Merci,
    Matthias
    Das Leben ist binär - du bist eine 1, oder eine 0

  • Sry, ich würds nicht unbedingt funktional nennen ;)

    Also das Array mit erlaubten Sendern solltest du VOR Beginn der while Schleife laden.
    Gerade wenn das ein MySQL Select ist, willst du den sicherlich nicht jedesmal wiederholen.
    Des weiteren solltest du imap_body erst holen, NACHDEM der Sender verifiziert wurde.

    Doch, mysql_fetch_row ist genau das richtige.
    Musst halt das Array füllen

    Quellcode

    1. $result = mysql_query("SELECT emailaddress FROM allowed");
    2. while($row = mysql_fetch_row($result)) {
    3. $mysearch[] = $row[0];
    4. }


    .. und versuch mal deinen Code formatiert zu lassen ;)