Projet

Général

Profil

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

Ajouté par Joël Cuissinat il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Début:
28/04/2020
Echéance:
% réalisé:

100%

Temps estimé:
0.00 h
Restant à faire (heures):
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 Voir (72,8 ko) Joël Cuissinat, 28/04/2020 11:07


Demandes liées

Lié à 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é

Révisions associées

Révision ffcc96dc (diff)
Ajouté par Emmanuel GARETTE il y a presque 4 ans

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

Révision 3b6d4597 (diff)
Ajouté par Emmanuel GARETTE il y a presque 4 ans

warning_only est une string (ref #30008)

Historique

#1 Mis à jour par Joël Cuissinat il y a presque 4 ans

#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

Formats disponibles : Atom PDF