Project

General

Profile

Development #71527

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

Added by Frédéric Péters 10 months ago. Updated about 2 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
21 November 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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.

Associated revisions

Revision 4b5a0eef (diff)
Added by Frédéric Péters about 2 months ago

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

History

#1

Updated by Guillaume Baffoin 10 months ago

Ç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

Updated by Frédéric Péters 6 months ago

  • Assignee set to Guillaume Baffoin
#7

Updated by Benjamin Dauvergne 6 months ago

Ç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

Updated by Frédéric Péters 6 months ago

  • Project changed from Admin système to Hobo
  • Assignee deleted (Guillaume Baffoin)
#9

Updated by Frédéric Péters 6 months ago

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

Updated by Benjamin Dauvergne 6 months ago

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

#11

Updated by Robot Gitea 2 months ago

  • Status changed from Nouveau to En cours
  • Assignee set to Frédéric Péters

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

#12

Updated by Robot Gitea 2 months ago

  • Status changed from En cours to Solution proposée
#13

Updated by Robot Gitea 2 months ago

  • Status changed from Solution proposée to Solution validée

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

#14

Updated by Robot Gitea about 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)

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

#15

Updated by Transition automatique about 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF