jellyfin/jellycon
1
0
Fork 0
mirror of https://github.com/jellyfin/jellycon.git synced 2025-02-22 23:03:06 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Don't use requests.json() if we can help it

Browse Source
This commit is contained in:
Matt 2022-07-08 21:44:58 -04:00
parent 921628b2a1
commit 276b18650c
1 changed files with 17 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
resources/lib/jellyfin.py
View File

@ -4,6 +4,7 @@ import xbmcaddon
from kodi_six.utils import py2_decode
import requests
import json
from .utils import get_device_id, get_version, load_user_details
from .lazylogger import LazyLogger
@ -36,7 +37,17 @@ class API:
r = requests.get(url, headers=self.headers, verify=self.verify_cert)
try:
response_data = r.json()
try:
'''
The requests library defaults to using simplejson to handle
json decoding. On low power devices and using Py3, this is
significantly slower than the builtin json library. Skip that
and just parse the json ourselves. Fall back to using
requests/simplejson if there's a parsing error.
'''
response_data = json.loads(r.text)
except ValueError:
response_data = r.json()
except:
response_data = {}
return response_data
@ -49,7 +60,11 @@ class API:
r = requests.post(url, json=payload, headers=self.headers, verify=self.verify_cert)
try:
response_data = r.json()
try:
# Much faster on low power devices, see above comment
response_data = json.loads(r.text)
except ValueError:
response_data = r.json()
except:
response_data = {}
return response_data