Project

General

Profile

Development #29240

Avoir un écran pour gérer le paramétrage de debug

Added by Benjamin Dauvergne 6 months ago. Updated 2 days ago.

Status:
Solution proposée
Priority:
Normal
Category:
-
Start date:
20 Dec 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Idée sur #29239:

Quand à DEBUG=True, je serai assez pour avoir un setting DEBUG_IP="<ip1> <ip2>", idem géré par hobo (je clique sur activate debug et ça ajoute mon ip cliente dans le truc), et avoir un middleware qui quand il voit l'IP pose DEBUG=True pour la durée de la requête.

Via #28930 ce serait gérable via un écran dans hobo :

[ Activer l'affichage des traces pour mon IP ]
IPs voyant les traces actuellement : 123.34.45.56 (La Ciotat, France), ...

Il faudrait peut-être expirer cette donnée régulièrement (genre ne pas laisser les traces activées plus de 8h).

0002-urls-coding-style-29240.patch View (3.09 KB) Benjamin Dauvergne, 08 Mar 2019 03:15 AM

0001-environment-coding-style-29240.patch View (21.2 KB) Benjamin Dauvergne, 08 Mar 2019 03:15 AM

0003-franceconnect-remove-dead-import-29240.patch View (843 Bytes) Benjamin Dauvergne, 08 Mar 2019 03:15 AM

0005-environment-factorize-get_setting_variable-from-fran.patch View (2.52 KB) Benjamin Dauvergne, 08 Mar 2019 03:15 AM

0008-add-debug-application-29240.patch View (9.85 KB) Benjamin Dauvergne, 08 Mar 2019 03:15 AM

0006-tests-coding-style-29240.patch View (3.56 KB) Benjamin Dauvergne, 08 Mar 2019 03:15 AM

0004-environment-rename-get_setting_variable-to-get_varia.patch View (2.19 KB) Benjamin Dauvergne, 08 Mar 2019 03:15 AM

0007-environment-add-a-setter-for-Variable.json-29240.patch View (1.16 KB) Benjamin Dauvergne, 08 Mar 2019 03:15 AM

0002-urls-coding-style-29240.patch View (3.09 KB) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

0001-environment-coding-style-29240.patch View (21.2 KB) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

0003-franceconnect-remove-dead-import-29240.patch View (843 Bytes) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

0005-environment-factorize-get_setting_variable-from-fran.patch View (2.52 KB) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

0008-add-debug-application-29240.patch View (9.85 KB) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

0009-reset-MUST_NOTIFY-on-request-start-29240.patch View (1.23 KB) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

0006-tests-coding-style-29240.patch View (3.56 KB) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

0004-environment-rename-get_setting_variable-to-get_varia.patch View (2.19 KB) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

0007-environment-add-a-setter-for-Variable.json-29240.patch View (1.16 KB) Benjamin Dauvergne, 08 Mar 2019 04:05 PM

Firefox_Screenshot_2019-03-13T16-19-52.276Z.png View (13.7 KB) Benjamin Dauvergne, 13 Mar 2019 05:20 PM

Firefox_Screenshot_2019-03-13T16-19-26.216Z.png View (14.4 KB) Benjamin Dauvergne, 13 Mar 2019 05:20 PM

0002-urls-coding-style-29240.patch View (3.09 KB) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0001-environment-coding-style-29240.patch View (21.2 KB) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0003-franceconnect-remove-dead-import-29240.patch View (843 Bytes) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0005-environment-factorize-get_setting_variable-from-fran.patch View (2.52 KB) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0008-add-debug-application-29240.patch View (10.8 KB) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0009-reset-MUST_NOTIFY-on-request-start-29240.patch View (1.23 KB) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0006-tests-coding-style-29240.patch View (3.56 KB) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0004-environment-rename-get_setting_variable-to-get_varia.patch View (2.19 KB) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0007-environment-add-a-setter-for-Variable.json-29240.patch View (1.16 KB) Benjamin Dauvergne, 13 Mar 2019 06:38 PM

0002-urls-coding-style-29240.patch View (3.09 KB) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0001-environment-coding-style-29240.patch View (21.3 KB) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0003-franceconnect-remove-dead-import-29240.patch View (843 Bytes) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0005-environment-factorize-get_setting_variable-from-fran.patch View (2.52 KB) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0008-add-debug-application-29240.patch View (10.8 KB) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0009-reset-MUST_NOTIFY-on-request-start-29240.patch View (1.23 KB) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0006-tests-coding-style-29240.patch View (3.56 KB) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0004-environment-rename-get_setting_variable-to-get_varia.patch View (2.19 KB) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0007-environment-add-a-setter-for-Variable.json-29240.patch View (1.01 KB) Benjamin Dauvergne, 18 Mar 2019 04:07 PM

0002-urls-coding-style-29240.patch View (3.26 KB) Benjamin Dauvergne, 24 May 2019 11:05 AM

0001-environment-coding-style-29240.patch View (21.3 KB) Benjamin Dauvergne, 24 May 2019 11:05 AM

0003-franceconnect-remove-dead-import-29240.patch View (843 Bytes) Benjamin Dauvergne, 24 May 2019 11:05 AM

0005-environment-factorize-get_setting_variable-from-fran.patch View (2.52 KB) Benjamin Dauvergne, 24 May 2019 11:05 AM

0008-add-debug-application-29240.patch View (10.8 KB) Benjamin Dauvergne, 24 May 2019 11:05 AM

0009-reset-MUST_NOTIFY-on-request-start-29240.patch View (1.23 KB) Benjamin Dauvergne, 24 May 2019 11:05 AM

0006-tests-coding-style-29240.patch View (3.56 KB) Benjamin Dauvergne, 24 May 2019 11:05 AM

0004-environment-rename-get_setting_variable-to-get_varia.patch View (2.19 KB) Benjamin Dauvergne, 24 May 2019 11:05 AM

0007-environment-add-a-setter-for-Variable.json-29240.patch View (1.01 KB) Benjamin Dauvergne, 24 May 2019 11:05 AM

0007-tests-coding-style-29240.patch View (3.57 KB) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0004-franceconnect-remove-dead-import-29240.patch View (845 Bytes) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0003-urls-coding-style-29240.patch View (3.26 KB) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0002-test_schemas-adapt-to-newly-exposed-field-of-Authent.patch View (903 Bytes) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0009-add-debug-application-29240.patch View (10.8 KB) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0001-environment-coding-style-29240.patch View (21.3 KB) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0005-environment-rename-get_setting_variable-to-get_varia.patch View (2.19 KB) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0008-environment-add-a-setter-for-Variable.json-29240.patch View (1.01 KB) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0010-reset-MUST_NOTIFY-on-request-start-29240.patch View (1.58 KB) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

0006-environment-factorize-get_setting_variable-from-fran.patch View (2.52 KB) Benjamin Dauvergne, 07 Jun 2019 10:20 PM

32593
32594

Related issues

Related to Hobo - Development #29149: Revoir le fonctionnement des logs de debug Solution proposée 17 Dec 2018

History

#1 Updated by Benjamin Dauvergne 3 months ago

  • Subject changed from pour avoir activer les écrans de trace pour son IP depuis hobo to Avoir un écran pour gérer le paramétrage de debug

#2 Updated by Benjamin Dauvergne 3 months ago

  • Assignee set to Benjamin Dauvergne

#5 Updated by Frédéric Péters 3 months ago

Du bout "je clique sur activate debug et ça ajoute mon ip cliente dans le truc" de la description, je me dis que ça peut être bien d'avoir ainsi l'ajout de son IP, sans avoir à la chercher. (peut-être que ça peut juste être un truc d'affichage, sur la page "votre adresse IP est : ").

#9 Updated by Benjamin Dauvergne 2 months ago

#11 Updated by Benjamin Dauvergne 10 days ago

Souci nouveau à cause des tests sur cook, une des mes modifs non essentielles :

# 0001-environment-coding-style-29240.patch
    def as_dict(self):
         as_dict = dict([(x, y) for (x, y) in self.__dict__.items()
-                        if type(y) in (int, str, unicode)])-
+                        if isinstance(y, six.integer_types + six.string_types)])
         as_dict['base_url'] = self.get_base_url_path()
         as_dict['service-id'] = self.Extra.service_id
         as_dict['service-label'] = force_text(self.Extra.service_label)

fait que des attributs non exposés dans le hobo.json des services le sont maintenant :

>       assert dump == expected_dump
E       assert '{\n    "prof...89"\n    }\n}' == '{\n    "profi...89"\n    }\n}'
E         Skipping 5532 identical leading characters in diff, use -v to show
E         -          "use_as_idp_for_self": true,
E         -             "variables": {}
E         ? ---
E         +          "variables": {}
E                   },
E                   {...
E         
E         ...Full output truncated (66 lines hidden), use '-vv' to show

Ici "use_as_idp_for_self" qui est booléen et désormais pris en compte par six.integer_types (les booléens sont des entiers en python pour des raisons historiques), je défais ça ?

#13 Updated by Nicolas Roche 2 days ago

Juste pour commenter le point ci-dessus avec les tests sur cook :
le test ne fait rien de plus que d'avertir quand l'environnement (hobo.json) évolue.

S'il évolue et que c'est souhaité, alors le patch 2 fait l'affaire :
- soit modifier le json attendu à la main
- soit le régénérer comme indiqué dans tests_schema/test_cook.py::test_cook_example

Je répond à côté car je n'ai pas d'avis sur le fait d'avoir une nouvelle entrée "use_as_idp_for_self".

Also available in: Atom PDF