Jump to content
Froxlor Forum

Question

Guten Morgen,

mir ist in Froxlor ein kleiner Fehler in Bezug auf CatchAll Email aufgefallen.

Zum System:

  • Debian 10, 64-Bit
  • Froxlor 0.10.11
  • Postfix, Dovecot, Spamc

Zum Problem:

Beim Anlegen einer neuen E-Mailadresse kann man je Domain nur eine CatchAll-Adresse definieren. Wenn man jedoch eine bestehende E-Mailadresse bearbeitet, kann man dort ganz einfach auf CatchAll umschalten.
Dadurch kann es passieren, dass mehrere Emailadressen unterhalb einer Domain als CatchAll definiert sind. Das ist mir erst aufgefallen, als ich E-Mails in meinem Postfach hatte, welche nicht für meine E-Mailadresse bestimmt waren.

Da auf dem Account in Froxlor mehrer Nutzer zugreifen (Feuerwehr) und E-Mailadressen hinzufügen, ändern und auch löschen - habe ich erst sehr spät gesehen das mehrere CatchAll vorhanden waren.

Wenn man eine Mail an eine E-Mailadresse sendet, welche als CatchAll definiert wurde, wird die Mail auch an alle anderen E-Mailadressen weitergeleitet, die unterhalb der Domain als CatchAll definiert sind.(Jedenfalls bei postfix)

Temporäre Lösung:

Damit eingehende Mails an eine vorhandene E-Mailadresse, welche als CatchAll definiert ist, auch nur im Postfach dieser E-Mailadresse landen, habe ich die SQL-Anweisung in der "mysql-virtual_alias_maps.cf" wie folgt abgeändert:

query = SELECT destination FROM mail_virtual AS v, panel_customers AS c WHERE c.customerid = v.customerid AND c.deactivated = 0 AND (v.email = '%s' OR v.email_full = '%s') AND trim(v.destination) <> ''

Ich habe das eben auch noch einmal in der Demoversion ausprobiert, Screenshot im Anhang.

 

Gruß

froxlor_catchall.png

Share this post


Link to post
Share on other sites

2 answers to this question

Recommended Posts

  • 1

Hey, danke fürs checken. Probierst du mal folgenden Patch?

diff --git a/lib/Froxlor/Api/Commands/Emails.php b/lib/Froxlor/Api/Commands/Emails.php
index 80270e55..af1cd0a2 100644
--- a/lib/Froxlor/Api/Commands/Emails.php
+++ b/lib/Froxlor/Api/Commands/Emails.php
@@ -106,10 +106,12 @@ class Emails extends \Froxlor\Api\ApiCommand implements \Froxlor\Api\ResourceEnt
                        );
                        $email_check = Database::pexecute_first($stmt, $params, true, true);
 
-                       if (strtolower($email_check['email_full']) == strtolower($email_full)) {
-                               \Froxlor\UI\Response::standard_error('emailexistalready', $email_full, true);
-                       } elseif ($email_check['email'] == $email) {
-                               \Froxlor\UI\Response::standard_error('youhavealreadyacatchallforthisdomain', '', true);
+                       if ($email_check) {
+                               if (strtolower($email_check['email_full']) == strtolower($email_full)) {
+                                       \Froxlor\UI\Response::standard_error('emailexistalready', $email_full, true);
+                               } elseif ($email_check['email'] == $email) {
+                                       \Froxlor\UI\Response::standard_error('youhavealreadyacatchallforthisdomain', '', true);
+                               }
                        }
 
                        $stmt = Database::prepare("
@@ -233,6 +235,19 @@ class Emails extends \Froxlor\Api\ApiCommand implements \Froxlor\Api\ResourceEnt
                        $iscatchall = '1';
                        $email_parts = explode('@', $result['email_full']);
                        $email = '@' . $email_parts[1];
+                       // catchall check
+                       $stmt = Database::prepare("
+                               SELECT `email_full` FROM `" . TABLE_MAIL_VIRTUAL . "`
+                               WHERE `email` = :email AND `customerid` = :cid AND `iscatchall` = '1'
+                       ");
+                       $params = array(
+                               "email" => $email,
+                               "cid" => $customer['customerid']
+                       );
+                       $email_check = Database::pexecute_first($stmt, $params, true, true);
+                       if ($email_check) {
+                               \Froxlor\UI\Response::standard_error('youhavealreadyacatchallforthisdomain', '', true);
+                       }
                } else {
                        $iscatchall = '0';
                        $email = $result['email_full'];

 

Share this post


Link to post
Share on other sites
  • 0

Hey, läuft.

Der Patch ist drin und nun ist es auch beim Bearbeiten einer E-Mailadresse nicht mehr möglich mehr als eine CatchAll je Domain zu definieren.
Ich habe dann zum Test einmal die eigentliche CatchAll deaktiviert und danach wieder aktiviert. Alles top!

Vielen Dank für die schnelle Reaktion.

 

Gruß

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

  • Similar Content

    • By tmuecksch
      Hi,
      seit gestern um 18:40h erhalte ich alle 5 Minuten folgende Mail. Da ich seit Längerem nichts am Server geändert habe (außer gelegentliche Update) würde ich mal ganz lapidar einen Fehler meinerseits ausschließen. Bisher hat ja alles problemlos geklappt.
      Nach etwas Recherche scheint der Fehler mit acme.sh zusammenzuhängen.
      Betreff: 
       
      Inhalt:
       
       
      Was kann ich tun um den Fehler zu Debugger und/oder zu beheben?
       
       
    • By thunfischbaum
      Hi everyone,
      just a little problem here. I have severa domains registered on my froxlor installation and a different hostname of course. So let's say domain1.de is my hostname. The second domain, which is used by a customer is called domain2.de . When I view the mail.log I am not able to find my connection request. I do only see the bruteforce bots with example@domain1.de connecting. But any example@domain2.de request is not getting forwareded. I did configure the installation accordingdly with the provided templates.
      Best regards,
      thunfischbaum
    • By thunfischbaum
      Hi everyone,
      I'm running a Debian 10 Buster Server with PHP7.3 installed. The PHP7.3 update is the new component in the system. since then, I can't find any Mailboxes. There are no folders in the var/customers/mail/[customer] directory.
      If you take look at the Screenshot, you see, that the E-Mail adress is existent in Froxlor. 

      Mail Bounce
      Mail Log
       
      If I missed a log, please let me know, so I can attach it
      Best regards,
      thunfischbaum

    • By JavaLink
      Hi
      I try install Froxlor several times bu every time i get error
       SQLSTATE[HY000] [1045] Access denied for user 'myuser'@'localhost' (using password: YES)
      I do these things for create use:
      CREATE USER 'myuser'@'localhost' IDENTIFIED BY 'pP123!@#Hello'; then
      create database froxlor;  
      and then
      GRANT ALL PRIVILEGES ON `froxlor` TO 'myuser'@'localhost';
       
      FLUSH PRIVILEGES;  
      I check connection with a file called con.php contain this ---> and it's OK.
      <?php $servername = "localhost"; $username = "myuser"; $password = "pP123!@#Hello"; // Create connection $conn = new mysqli($servername, $username, $password); // Check connection if ($conn->connect_error) { die("Connection failed: " . $conn->connect_error); } echo "Connected successfully";  
      but after run froxlor to install it and fill all information, i get that error.
      Also i check again with server and find out that the `myuser` user is deleted...What happened? What Froxlor do?
    • Guest
      By Guest
      in der config-Datei steht
       
      was für eine valid e-mail Adresse muss das sein ?  diese root@frx.beispiel.de ist ja bisher nicht eingerichtet und packt systemmeldungen in ein verzeichnis

      sollte dort also eine mailadresse rein, die auf einem anderen Server liegt und Mails empfängt, oder sollte/könnte diese root@beispiel.de im froxlor eingerichtet sein ?
×
×
  • Create New...