wscalls: do not log/notify about app errors anymore (#87554) #1218

Merged
fpeters merged 1 commits from wip/87554-wscall-do-no-record-app-errors into main 2024-03-06 18:10:04 +01:00
Owner
No description provided.
fpeters reviewed 2024-03-03 11:14:48 +01:00
@ -63,6 +74,7 @@ class NamedWsCallUI:
'record_on_errors',
title=_('Record on errors'),
value=self.wscall.record_on_errors if self.wscall.slug else True,
default_value=True,
Author
Owner

Détail vu en passant, pour que l'onglet "Gestion des erreurs" ne soit pas marqué comme contenant une modification dès le début.

Détail vu en passant, pour que l'onglet "Gestion des erreurs" ne soit pas marqué comme contenant une modification dès le début.
@ -565,6 +565,7 @@ class HtmlWidget:
self.attrs = {}
self.string = string
self.title = title
self.tab = kwargs.pop('tab', None)
Author
Owner

Visiblement c'est la première fois qu'on met un HtmlWidget dans un onglet.

Visiblement c'est la première fois qu'on met un HtmlWidget dans un onglet.
wcs/wscalls.py Outdated
@ -208,3 +208,2 @@
app_error_code = get_app_error_code(response, data, 'json')
if (app_error_code != 0 or status >= 400) and (notify_on_errors or record_on_errors):
if status >= 400 and (notify_on_errors or record_on_errors):
record_wscall_error(status, data, response, app_error_code, notify_on_errors, record_on_errors)
Author
Owner

Soit on est dans le cas "appel webservice nommé" et on n'enregistre plus les erreurs applicatives (l'objet du ticket); soit pour les appels webservices dans les workflows on passe déjà notify_on_errors=False et record_on_errors=False donc on n'entrait pas là-dedans.

Soit on est dans le cas "appel webservice nommé" et on n'enregistre plus les erreurs applicatives (l'objet du ticket); soit pour les appels webservices dans les workflows on passe déjà notify_on_errors=False et record_on_errors=False donc on n'entrait pas là-dedans.
fpeters force-pushed wip/87554-wscall-do-no-record-app-errors from 7193afefc8 to fafee7c11a 2024-03-03 11:20:57 +01:00 Compare
fpeters force-pushed wip/87554-wscall-do-no-record-app-errors from fafee7c11a to 469b5d9bb9 2024-03-03 11:43:55 +01:00 Compare
fpeters changed title from WIP: wscalls: do not log/notify about app errors anymore (#87554) to wscalls: do not log/notify about app errors anymore (#87554) 2024-03-03 11:50:31 +01:00
lguerin approved these changes 2024-03-06 09:02:58 +01:00
fpeters merged commit 8c26581924 into main 2024-03-06 18:10:04 +01:00
fpeters deleted branch wip/87554-wscall-do-no-record-app-errors 2024-03-06 18:10:04 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1218
No description provided.