Tâche #30008
Scénario #29823: Ajouter une variable pour configurer la taille limite de message dans Exim
Le "warning" sur "exim_message_size_limit" n'en est pas un
100%
Description
Testé en 2.7.1 et 2.7.2 dans GenConfig (cf. copie d'écran) et ligne de commande :
root@scribe:~# CreoleSet exim_message_size_limit 999 Valeur invalide : "999" est une valeur invalide pour l'option "Taille maximale d'un message à envoyer en Mo" de type nombre, La valeur est supérieur à la valeur de php_upload_max_filesize. Elle ne sera pas prise en compte et php_upload_max_filesize sera prise en compte par le webmail lors de l'envoi. root@scribe:~# CreoleGet exim_message_size_limit 10
Demandes liées
Révisions associées
support de warning dans tous les cas dans un check (ref #30008)
warning_only est une string (ref #30008)
Historique
#1 Mis à jour par Joël Cuissinat il y a presque 4 ans
- Fichier exim_message_size_limit.png Voir ajouté
#2 Mis à jour par Emmanuel GARETTE il y a presque 4 ans
- Statut changé de Nouveau à En cours
#3 Mis à jour par Emmanuel GARETTE il y a presque 4 ans
- Statut changé de En cours à Résolu
- Assigné à mis à Emmanuel GARETTE
- % réalisé changé de 0 à 100
#4 Mis à jour par Joël Cuissinat il y a presque 4 ans
OK
root@scribe:~# CreoleSet exim_message_size_limit 999 IntOption:0: ValueWarning: attention, "999" peut être une option de type nombre invalide pour "Taille maximale d'un message à envoyer en Mo", La valeur est supérieur à la valeur de php_upload_max_filesize. Elle ne sera pas prise en compte et php_upload_max_filesize sera prise en compte par le webmail lors de l'envoi. root@scribe:~# CreoleGet exim_message_size_limit 999 999
~/git/creole$ git branch -r --contains ffcc96dc origin/2.7.1/master origin/2.7.2/master origin/HEAD -> origin/master origin/dist/eole/2.7.1/master origin/dist/eole/2.7.2/master origin/dist/eole/2.8.0/master origin/master
Au passage, ça pourrait peut-être régler : #28397#note-6 ?
#5 Mis à jour par Joël Cuissinat il y a presque 4 ans
- Lié à Scénario #28397: Ajouter des tests de validation sur la variable "Nom DNS du réseau local" (nom_domaine_local) ajouté
#6 Mis à jour par Joël Cuissinat il y a presque 4 ans
- Statut changé de Résolu à Fermé
- Restant à faire (heures) mis à 0.0
#7 Mis à jour par Joël Cuissinat il y a presque 4 ans
- Statut changé de Fermé à En cours
- Temps estimé mis à 0.00 h
C'est fonctionnel dans le cadre de cette demande mais avec cette modification, les tests unitaires signalent des changements de comportement qui me semblent anormaux https://dev-eole.ac-dijon.fr/jenkins/job/2.7.1/job/test-moduletests-eolebase-2.7.1-amd64/323/parsed_console/ :
root@eolebase:~# py.test /usr/share/creole/tests/test_genconfig.py def test_wrong_calc(): """ pas de traceback en cas de valeur calculée invalide """ config = _load('wrong_calc') assert config.creole.test.test_value == 'FR' assert config.creole.test.test_country_name == 'FR' assert config.creole.test.test_country_name2 == 'FR' lib.set_value(ID_, 'test', 'test_value', 'EU') assert config.creole.test.test_country_name == 'EU' assert config.creole.test.test_country_name2 == 'EU' lib.set_value(ID_, 'test', 'test_value', 'I2') var = lib.get_variable(ID_, 'test', 'test_country_name') > assert var['value'] is None E AssertionError: assert 'I2' is None test_genconfig.py::test_wrong_calc UnicodeOption:0: ValueWarning: attention, "I2" peut être une option de type texte unicode invalide pour "pas de problème avec "fr" et "eu" mais problème "i2"", La valeur doit être composée uniquement de lettres majuscules
Étrangement le second test en erreur (test_check_variable du fichier test_loader.py) ne l'est plus si on l'exécute individuellement, il y aurait un effet de bord avec des tests précédents ?
cfg = loader.creole_loader(load_values=False, load_extra=True, rw=True) > raises(ValueError, "cfg.creole.general.int = 150") E Failed: DID NOT RAISE <type 'exceptions.ValueError'>
#8 Mis à jour par Joël Cuissinat il y a presque 4 ans
- Statut changé de En cours à Résolu
#9 Mis à jour par Joël Cuissinat il y a presque 4 ans
- Description mis à jour (diff)
#10 Mis à jour par Joël Cuissinat il y a presque 4 ans
- Statut changé de Résolu à Fermé
Les tests unitaires sont de nouveau OK et une valeur trop élevée entraîne bien un warning :
root@scribe:~# CreoleSet exim_message_size_limit 999 IntOption:0: ValueWarning: attention, "999" peut être une option de type nombre invalide pour "Taille maximale d'un message à envoyer en Mo", La valeur est supérieure à celle de php_upload_max_filesize. Elle ne sera pas prise en compte et php_upload_max_filesize sera prise en compte par le webmail lors de l'envoi. root@scribe:~# CreoleGet exim_message_size_limit 999