git subrepo commit (merge) mailcow/src/mailcow-dockerized

subrepo: subdir:   "mailcow/src/mailcow-dockerized"
  merged:   "02ae5285"
upstream: origin:   "https://github.com/mailcow/mailcow-dockerized.git"
  branch:   "master"
  commit:   "649a5c01"
git-subrepo: version:  "0.4.3"
  origin:   "???"
  commit:   "???"
Change-Id: I870ad468fba026cc5abf3c5699ed1e12ff28b32b
diff --git a/mailcow/src/mailcow-dockerized/data/web/inc/functions.dkim.inc.php b/mailcow/src/mailcow-dockerized/data/web/inc/functions.dkim.inc.php
index 2a19ff0..33ee49f 100644
--- a/mailcow/src/mailcow-dockerized/data/web/inc/functions.dkim.inc.php
+++ b/mailcow/src/mailcow-dockerized/data/web/inc/functions.dkim.inc.php
@@ -1,8 +1,8 @@
 <?php

 

 function dkim($_action, $_data = null, $privkey = false) {

-	global $redis;

-	global $lang;

+  global $redis;

+  global $lang;

   switch ($_action) {

     case 'add':

       if ($_SESSION['mailcow_cc_role'] != "admin") {

@@ -13,7 +13,7 @@
         );

         return false;

       }

-      $key_length	= intval($_data['key_size']);

+      $key_length = intval($_data['key_size']);

       $dkim_selector = (isset($_data['dkim_selector'])) ? $_data['dkim_selector'] : 'dkim';

       $domains = array_map('trim', preg_split( "/( |,|;|\n)/", $_data['domains']));

       $domains = array_filter($domains);

@@ -21,7 +21,7 @@
         if (!is_valid_domain_name($domain) || !is_numeric($key_length)) {

           $_SESSION['return'][] = array(

             'type' => 'danger',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('dkim_domain_or_sel_invalid', $domain)

           );

           continue;

@@ -29,7 +29,7 @@
         if ($redis->hGet('DKIM_PUB_KEYS', $domain)) {

           $_SESSION['return'][] = array(

             'type' => 'danger',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('dkim_domain_or_sel_invalid', $domain)

           );

           continue;

@@ -37,7 +37,7 @@
         if (!ctype_alnum(str_replace(['-', '_'], '', $dkim_selector))) {

           $_SESSION['return'][] = array(

             'type' => 'danger',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('dkim_domain_or_sel_invalid', $domain)

           );

           continue;

@@ -62,7 +62,7 @@
           catch (RedisException $e) {

             $_SESSION['return'][] = array(

               'type' => 'danger',

-              'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+              'log' => array(__FUNCTION__, $_action, $_data),

               'msg' => array('redis_error', $e)

             );

             continue;

@@ -76,7 +76,7 @@
             catch (RedisException $e) {

               $_SESSION['return'][] = array(

                 'type' => 'danger',

-                'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+                'log' => array(__FUNCTION__, $_action, $_data),

                 'msg' => array('redis_error', $e)

               );

               continue;

@@ -84,14 +84,14 @@
           }

           $_SESSION['return'][] = array(

             'type' => 'success',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('dkim_added', $domain)

           );

         }

         else {

           $_SESSION['return'][] = array(

             'type' => 'danger',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('dkim_domain_or_sel_invalid', $domain)

           );

           continue;

@@ -102,7 +102,7 @@
       if ($_SESSION['mailcow_cc_role'] != "admin") {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => 'access_denied'

         );

         return false;

@@ -112,7 +112,7 @@
       if (empty($from_domain_dkim)) {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => array('dkim_domain_or_sel_invalid', $from_domain)

         );

         continue;

@@ -128,14 +128,14 @@
         catch (RedisException $e) {

           $_SESSION['return'][] = array(

             'type' => 'danger',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('redis_error', $e)

           );

           continue;

         }

         $_SESSION['return'][] = array(

           'type' => 'success',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => array('dkim_duplicated', $from_domain, $to_domain)

         );

       }

@@ -144,7 +144,7 @@
       if ($_SESSION['mailcow_cc_role'] != "admin") {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => 'access_denied'

         );

         return false;

@@ -156,7 +156,7 @@
       if ($ssl_error = openssl_error_string()) {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => array('private_key_error', $ssl_error)

         );

         return false;

@@ -167,13 +167,13 @@
       array_shift($pem_public_key_array);

       array_pop($pem_public_key_array);

       // Implode as single string

-      $pem_public_key = implode('', $pem_public_key_array);

+      $pem_public_key = implode('', (array)$pem_public_key_array);

       $dkim_selector = (isset($_data['dkim_selector'])) ? $_data['dkim_selector'] : 'dkim';

-      $domain	= $_data['domain'];

+      $domain = $_data['domain'];

       if (!is_valid_domain_name($domain)) {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => array('dkim_domain_or_sel_invalid', $domain)

         );

         return false;

@@ -182,7 +182,7 @@
         if ($overwrite_existing == 0) {

           $_SESSION['return'][] = array(

             'type' => 'danger',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('dkim_domain_or_sel_exists', $domain)

           );

           return false;

@@ -191,13 +191,13 @@
       if (!ctype_alnum($dkim_selector)) {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => array('dkim_domain_or_sel_invalid', $domain)

         );

         return false;

       }

       try {

-        dkim('delete', $domain);

+        dkim('delete', (array)$domain);

         $redis->hSet('DKIM_PUB_KEYS', $domain, $pem_public_key);

         $redis->hSet('DKIM_SELECTORS', $domain, $dkim_selector);

         $redis->hSet('DKIM_PRIV_KEYS', $dkim_selector . '.' . $domain, $private_key_normalized);

@@ -205,7 +205,7 @@
       catch (RedisException $e) {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => array('redis_error', $e)

         );

         return false;

@@ -218,14 +218,14 @@
       catch (RedisException $e) {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => array('redis_error', $e)

         );

         return false;

       }

       $_SESSION['return'][] = array(

         'type' => 'success',

-        'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+        'log' => array(__FUNCTION__, $_action, $_data),

         'msg' => array('dkim_added', $domain)

       );

       return true;

@@ -251,7 +251,7 @@
         }

         if ($GLOBALS['SPLIT_DKIM_255'] === true) {

           $dkim_txt_tmp = str_split('v=DKIM1;k=rsa;t=s;s=email;p=' . $redis_dkim_key_data, 255);

-          $dkimdata['dkim_txt'] = sprintf('"%s"', implode('" "', $dkim_txt_tmp ) );

+          $dkimdata['dkim_txt'] = sprintf('"%s"', implode('" "', (array)$dkim_txt_tmp ) );

         }

         else {

           $dkimdata['dkim_txt'] = 'v=DKIM1;k=rsa;t=s;s=email;p=' . $redis_dkim_key_data;

@@ -270,7 +270,7 @@
       if ($_SESSION['mailcow_cc_role'] != "admin") {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => 'access_denied'

         );

         return false;

@@ -286,7 +286,7 @@
       if ($_SESSION['mailcow_cc_role'] != "admin") {

         $_SESSION['return'][] = array(

           'type' => 'danger',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => 'access_denied'

         );

         return false;

@@ -295,7 +295,7 @@
         if (!is_valid_domain_name($domain)) {

           $_SESSION['return'][] = array(

             'type' => 'danger',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('dkim_domain_or_sel_invalid', $domain)

           );

           continue;

@@ -309,14 +309,14 @@
         catch (RedisException $e) {

           $_SESSION['return'][] = array(

             'type' => 'danger',

-            'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+            'log' => array(__FUNCTION__, $_action, $_data),

             'msg' => array('redis_error', $e)

           );

           continue;

         }

         $_SESSION['return'][] = array(

           'type' => 'success',

-          'log' => array(__FUNCTION__, $_action, $_data, $privkey),

+          'log' => array(__FUNCTION__, $_action, $_data),

           'msg' => array('dkim_removed', htmlspecialchars($domain))

         );

       }