AlternC / AlternC

AlternC Hosting Panel
https://alternc.com/Home-fr
GNU General Public License v2.0
107 stars 68 forks source link

Le nom du domaine est syntaxiquement incorrect #210

Open mistur opened 6 years ago

mistur commented 6 years ago

Bonjour,

suite à la dernière maj en 3.2.11 je ne peux plus éditer mes sous domaines :

sur les pages du type /dom_subedit.php?sub_domain_id=$ID

screenshot-2017-11-11 panneau de controle alternc

$ dpkg -l | grep alternc                                                                                                                                                                                                                                                                     ii  alternc                              3.2.11                            all          Hosting software managment suite for Debian
ii  alternc-api                          3.2.11                            all          Rest or get/post API for AlternC
ii  alternc-awstats                      3.2.11                            all          Awstats statistics module for AlternC
ii  alternc-mailman                      3.2                               all          Mailman module for AlternC
ii  alternc-roundcube                    3.2.11                            all          Roundcube-Webmail plugin and configuration for AlternC
ii  alternc-ssl                          3.2.11                            all          HTTPS module for AlternC
ii  mailman                              1:2.1.23-1~alternc.2              i386         Powerful, web-based mailing list manager

Yoann

camlafit commented 6 years ago

Hello

About first line what is your domain set ? About second line we have change $msg class, in next version (or nightly) should be solve

mistur commented 6 years ago

All the domains already installed are impacted.

mistur.org, www.mistur.org and test.mistur.org for example :

mysql> select * from domaines  where domaine like "mistur.org" ;
+--------+------------+--------+-------+---------+------------+------------+----+---------+
| compte | domaine    | gesdns | gesmx | noerase | dns_action | dns_result | id | zonettl |
+--------+------------+--------+-------+---------+------------+------------+----+---------+
|   2001 | mistur.org |      1 |     1 |       0 | OK         | 0          |  2 |     300 |
+--------+------------+--------+-------+---------+------------+------------+----+---------+
1 row in set (0.00 sec)
mysql> select * from sub_domaines  where domaine like "mistur.org" and ( sub = "www" or sub = "test" );
+-----+--------+------------+------+--------------------+-------+------------+------------+----------+
| id  | compte | domaine    | sub  | valeur             | type  | web_action | web_result | enable   |
+-----+--------+------------+------+--------------------+-------+------------+------------+----------+
|   4 |   2001 | mistur.org | www  | /mistur.org/www/   | VHOST | OK         | 0          | ENABLED  |
| 130 |   2001 | mistur.org | test | /mistur.org/litchi | VHOST | OK         | 0          | DISABLED |
| 543 |   2001 | mistur.org | test | 178.33.105.117     | ip    | OK         | 6          | ENABLED  |
+-----+--------+------------+------+--------------------+-------+------------+------------+----------+

I can't even delete an entrie

Best regards,

Yoann

mistur commented 6 years ago

l'erreur à lieu la : class/m_dom.php -> ligne 1176 ça fail sur checkfqdn($dom) car la variable $dom est vide j'ai modifié le message d'erreur :

        $t = checkfqdn($dom);
        if ($t) {
            $msg->raise("ERROR", "dom", _("The domain name is syntaxically incorrect 2 \"$dom\""));
            return false;
        }

ça renvoie ça :

The domain name is syntaxically incorrect 2 ""

soul9 commented 6 years ago

J'ai le même probleme a une mise a jour de 3.2.10 à 3.2.11.

soul9 commented 6 years ago

donc dans la ligne 62 de panel/admin/dom_subedit.php: if ($msg->has_msgs("ERROR")) { le problème est que le warning de déprécation du plugin mailman est détecté comme ERROR et donc fait sortir le script.

soul9 commented 6 years ago

la solution est de changer $msg->raise( "error","err","Deprecation warning: The old messaging class is still used by ".json_encode( $caller )); dans panel/class/m_err.php ligne 153 à $msg->raise( "info","err","Deprecation warning: The old messaging class is still used by ".json_encode( $caller )); Idéalement ca devrait pas etre affiché aux users parce qu'ils s'en foutent, ce devrait etre affiché que pour les admin.

mistur commented 6 years ago

Bonsoir, ça semble corriger le problème, je peux de nouveau éditer et supprimer des sous-domaines et le message d'erreur apparait toujours mais en vert :) Merci pour le patch Yoann

soul9 commented 6 years ago

Voila, j'ai encore eu ce probleme, cette fois ci dans dom_substatus.php. Sur les lignes 29 et 30 de https://github.com/AlternC/AlternC/blob/stable-3.1/bureau/admin/dom_substatus.php#L29 il faut mettre get en type, pas post, car la ligne 200 et 207 de dom_edit.php le met en get https://github.com/AlternC/AlternC/blob/stable-3.1/bureau/admin/dom_edit.php#L200