Project

General

Profile

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

Added by Joël Cuissinat over 2 years ago. Updated over 2 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
04/28/2020
Due date:
% Done:

100%

Estimated time:
0.00 h
Remaining (hours):
0.0

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

exim_message_size_limit.png View (72.8 KB) Joël Cuissinat, 04/28/2020 11:07 AM


Related issues

Related to EOLE AD DC - Scénario #28397: Ajouter des tests de validation sur la variable "Nom DNS du réseau local" (nom_domaine_local) Partiellement Réalisé

Associated revisions

Revision ffcc96dc (diff)
Added by Emmanuel GARETTE over 2 years ago

support de warning dans tous les cas dans un check (ref #30008)

Revision 3b6d4597 (diff)
Added by Emmanuel GARETTE over 2 years ago

warning_only est une string (ref #30008)

History

#2 Updated by Emmanuel GARETTE over 2 years ago

  • Status changed from Nouveau to En cours

#3 Updated by Emmanuel GARETTE over 2 years ago

  • Status changed from En cours to Résolu
  • Assigned To set to Emmanuel GARETTE
  • % Done changed from 0 to 100

#4 Updated by Joël Cuissinat over 2 years ago

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 Updated by Joël Cuissinat over 2 years ago

  • Related to Scénario #28397: Ajouter des tests de validation sur la variable "Nom DNS du réseau local" (nom_domaine_local) added

#6 Updated by Joël Cuissinat over 2 years ago

  • Status changed from Résolu to Fermé
  • Remaining (hours) set to 0.0

#7 Updated by Joël Cuissinat over 2 years ago

  • Status changed from Fermé to En cours
  • Estimated time set to 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 Updated by Joël Cuissinat over 2 years ago

  • Status changed from En cours to Résolu

#9 Updated by Joël Cuissinat over 2 years ago

  • Description updated (diff)

#10 Updated by Joël Cuissinat over 2 years ago

  • Status changed from Résolu to 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

Also available in: Atom PDF