Projet

Général

Profil

Development #71527

désactiver la levée d'erreur quand on est en maintenance ?

Ajouté par Frédéric Péters il y a plus d'un an. Mis à jour il y a 9 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
21 novembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Sans ça, on se tape une collection d'erreurs,

Service Unavailable: /api/users/synchronization/
Service Unavailable: /__services.js
Service Unavailable: /idp/saml2/metadata
etc.

Révisions associées

Révision 4b5a0eef (diff)
Ajouté par Frédéric Péters il y a 9 mois

misc: do not send error emails when in maintenance mode (#71527)

Historique

#1

Mis à jour par Guillaume Baffoin il y a plus d'un an

Ça, c'est dû au fait que les IPs de la plateforme n'étaient pas autorisées à communiquer entre elle malgré le mode maintenance.
suite à l'ajout de celles-ci, on a plus d'erreur.
(je croyais que la case "Désactiver les tâche routinières" permettait de ne pas avoir ce genre de souci malgré l'absence des ips de plateforme.)

#3

Mis à jour par Frédéric Péters il y a environ un an

  • Assigné à mis à Guillaume Baffoin
#7

Mis à jour par Benjamin Dauvergne il y a environ un an

Ça va être dans hobo pour une partie, par exemple désactiver les mails sur un log.error() mais sinon dans puppet pour désactiver sentry, modules/publik/templates/sentry.py .

#8

Mis à jour par Frédéric Péters il y a environ un an

  • Projet changé de Admin système à Hobo
  • Assigné à Guillaume Baffoin supprimé
#9

Mis à jour par Frédéric Péters il y a environ un an

De mon côté sur les logs je ne pourrais proposer que :

--- a/hobo/multitenant/log.py
+++ b/hobo/multitenant/log.py
@@ -15,10 +15,17 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.

 import django.utils.log
+from django.conf import settings
 from django.db import connection

 class AdminEmailHandler(django.utils.log.AdminEmailHandler):
+    def emit(self, record):
+        maintenance_mode = getattr(settings, 'MAINTENANCE_PAGE', None)
+        if maintenance_mode:
+            return
+        return super().emit(record)
+
     def format_subject(self, subject):
         from .models import Tenant

si c'est acceptable vis-à-vis de pratiques logging (vs je ne sais quoi ajouter des filtres et modifier LOGGING), je peux mettre ça dans un patch.

#10

Mis à jour par Benjamin Dauvergne il y a environ un an

C'est très bien pour la partie EmailHandler.

#11

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#12

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de En cours à Solution proposée
#13

Mis à jour par Robot Gitea il y a 9 mois

  • Statut changé de Solution proposée à Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#14

Mis à jour par Robot Gitea il y a 9 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#15

Mis à jour par Transition automatique il y a 9 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#16

Mis à jour par Transition automatique il y a 7 mois

Automatic expiration

Formats disponibles : Atom PDF