Development #71527
désactiver la levée d'erreur quand on est en maintenance ?
0%
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
Historique
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.)
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 .
Mis à jour par Frédéric Péters il y a environ un an
- Projet changé de Admin système à Hobo
- Assigné à
Guillaume Baffoinsupprimé
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.
Mis à jour par Benjamin Dauvergne il y a environ un an
C'est très bien pour la partie EmailHandler.
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 :
- URL : https://git.entrouvert.org/entrouvert/hobo/pulls/54
- Titre : WIP: misc: do not send error emails when in maintenance mode (#71527)
- Modifications : https://git.entrouvert.org/entrouvert/hobo/pulls/54/files
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 :
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 :
- URL : https://git.entrouvert.org/entrouvert/hobo/pulls/54
- Titre : misc: do not send error emails when in maintenance mode (#71527)
- Modifications : https://git.entrouvert.org/entrouvert/hobo/pulls/54/files
Mis à jour par Transition automatique il y a 9 mois
- Statut changé de Résolu (à déployer) à Solution déployée
misc: do not send error emails when in maintenance mode (#71527)