Bug #33788
error_record plante si appellé avec using=None
Début:
07 juin 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Sur prod pfwb ok mais en local :
Environment:
Request Method: GET
Request URL: http://127.0.0.1:8003/contact/
Django Version: 1.8.19
Python Version: 2.7.15
Installed Applications:
('django.contrib.contenttypes',
'django.contrib.auth',
'django.contrib.sessions',
'django.contrib.messages',
'django.contrib.staticfiles',
'django.contrib.humanize',
'watson',
'autocomplete_light',
'grappelli',
'django.contrib.admin',
'django_tables2',
'django_journal',
'crispy_forms',
'docbow_project.humantime',
'docbow_project.docbow',
'docbow_project.pfwb')
Installed Middleware:
('django.middleware.common.CommonMiddleware',
'django.contrib.sessions.middleware.SessionMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware',
'docbow_project.docbow.middleware.KeepUserAroundMiddleware',
'django_journal.middleware.JournalMiddleware',
'watson.middleware.SearchContextMiddleware')
Traceback:
File "/home/cazino/envs/docbow-env/local/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
132. response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/cazino/src/docbow/docbow_project/docbow/views.py" in contact
353. request.error_record('error', msg)
File "/home/cazino/src/django-journal/django_journal/middleware.py" in error_record
24. django_journal.error_record(tag, template, using=using, **kwargs)
File "/home/cazino/src/django-journal/django_journal/__init__.py" in error_record
75. **kwargs)
Exception Type: TypeError at /contact/
Exception Value: record() got multiple values for keyword argument 'using'
Fichiers
Révisions associées
make error_record resilient to None in 'using' arg (#33788)
Historique
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
Bougé de docbow django-journal : on tombe là dessus dans docbow si le groupe à contacter n'existe pas, mais c'est bien un problème django-journal que docbow appelle correctement.
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Sujet changé de Erreur 500 sur /contact/ (en local) à error_record crash si appel
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Fichier 0001-tests-improvements-33788.patch 0001-tests-improvements-33788.patch ajouté
- Fichier 0002-make-error_record-resilient-to-None-in-using-arg-337.patch 0002-make-error_record-resilient-to-None-in-using-arg-337.patch ajouté
- Sujet changé de error_record crash si appel à error_record plante si appellé avec using=None
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Description du bug dans le sujet, c'est le middleware de django_journal qui peut appeler error_record avec using=None : auto sabordage.
Premier patch les tests dans leur propre répertoire, coverage, postgres, etc.
Deuxième patch je corrige.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Emmanuel Cazenave
commit efd2da126fd2e43bc3d4ed517037fe1cf797f9e9
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Tue Jul 2 19:49:48 2019 +0200
make error_record resilient to None in 'using' arg (#33788)
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests improvements (#33788)
Move tests in their own directory, add code coverage, run tests using postgres.