Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(79)

Issue 435131003: trytond: Unify logging exception in wrappers (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 month, 3 weeks ago by ced
Modified:
3 weeks, 5 days ago
Reviewers:
rietveld-bot, reviewbot
Visibility:
Public.

Description

trytond: Unify logging exception in wrappers Only with_pool is responsible to log exceptions. Also it does not log UserError nor UserWarning but convert them as HTTP client error. issue11483

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -13 lines) Patch
M trytond/protocols/wrappers.py View 4 chunks +13 lines, -13 lines 0 comments Download

Messages

Total messages: 4
ced
1 month, 3 weeks ago (2022-05-13 10:34:08 UTC) #1
reviewbot
checks OK URL: https://codereview.tryton.org/435131003
1 month, 3 weeks ago (2022-05-13 10:55:28 UTC) #2
rietveld-bot_tryton.org
New changeset 013abca098a7 by Cédric Krier in branch 'default': Unify logging exception in wrappers https://hg.tryton.org/trytond/rev/013abca098a7
3 weeks, 5 days ago (2022-06-09 07:29:58 UTC) #3
rietveld-bot_tryton.org
3 weeks, 5 days ago (2022-06-09 07:30:03 UTC) #4
New changeset 0f527fe2c637 by Cédric Krier in branch 'default':
Unify logging exception in wrappers
https://hg.tryton.org/tryton-env/rev/0f527fe2c637
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld d9ca037-tainted