Proper error management for JSON-2 API

Author: florentx

CHANGES.rst

@@ -11,6 +11,8 @@ Changelog
 * Consider users with restricted rights.  Since Odoo 15, most
   users don't have access to ``ir.model``.
 
+* Enhance error management for JSON-2 API with Odoo >= 19.
+
 
 2.4.6 (2025-10-31)
 ~~~~~~~~~~~~~~~~~~

odooly.py

@@ -144,6 +144,9 @@ def request(self, url, *, method='POST', data=None, json=None, headers=None, **k
             resp.raise_for_status()
             return resp if method == 'HEAD' else resp.text if json is None else resp.json()
 
+        def _parse_error(self, error):
+            return error.response.status_code, error.response.json()
+
     else:  # urllib.request
         def __init__(self):
             self._session = build_opener(HTTPCookieProcessor(), HTTPSHandler(context=http_context))
@@ -160,6 +163,9 @@ def request(self, url, *, method='POST', data=None, json=None, headers=None, _js
             with self._session.open(request) as resp:
                 return resp if method == 'HEAD' else resp.read().decode() if json is None else _json.load(resp)
 
+        def _parse_error(self, error):
+            return error.code, json.load(error)
+
 
 def _memoize(inst, attr, value, doc_values=None):
     if hasattr(value, '__get__') and not hasattr(value, '__self__'):
@@ -551,7 +557,7 @@ class Json2:
     _doc_endpoint = '/doc-bearer'
 
     def __init__(self, client, database, api_key):
-        self._req = HTTPSession().request
+        self._http = HTTPSession()
         self._server = urljoin(client._server, '/')
         self._headers = {
             'Authorization': f'Bearer {api_key}',
@@ -594,10 +600,19 @@ def __call__(self, model, method, args, kw=None):
     def __repr__(self):
         return f"<Json2 '{self._server[:-1]}{self._endpoint}'>"
 
+    def _req(self, url, json, headers, method):
+        try:
+            return self._http.request(url, method=method, json=json, headers=headers)
+        except OSError as exc:
+            status_code, result = self._http._parse_error(exc)
+            if status_code == 422:  # UnprocessableEntity
+                raise ServerError({'code': status_code, 'data': result})
+            raise
+
     def _check(self, uid=None):
         url = urljoin(self._server, f'{self._endpoint}/res.users/context_get')
         try:
-            context = self._req(url, json={}, headers=self._headers)
+            context = self._http.request(url, json={}, headers=self._headers, method='POST')
         except (OSError, ServerError):
             return False
         return self if (not uid or uid == context['uid']) else False