Respect timeout from server

[glensc]

What is your feature request?

Add support for _server.timeout being used instead (or addition to) global TIMEOUT

---

PlexAPI may call reload() on property access, and that call is routed to server.query() which can only read the global TIMEOUT (because called with out timeout=xxx parameter):

     def _reload():
        data = self._server.query(key)

Seems the problem is that each query method takes own timeout parameter, or uses global:

• https://github.com/pkkid/python-plexapi/blob/a7d29a3638bdbec6e223c373a47aeb36a2e60bd5/plexapi/server.py

and the global comes from config:

• python-plexapi/plexapi/__init__.py

  Line 20 in a7d29a3

  TIMEOUT = CONFIG.get('plexapi.timeout', 30, int)

Reported in:

• add_to_collection : Plex ReadTimeout Taxel/PlexTraktSync#1455 (comment)

Are there any workarounds?

Overwite the global TIMEOUT in __init__.py. Ugly but likely will work.

Code Snippets

No response

Additional Context

No response

[JonnyWong16]

Probably easiest to change the __init__ to self.timeout = timeout or TIMEOUT, then query to timeout = timeout or self._timeout.

PR welcome.

[glensc]

__init__ to self.timeout = timeout or TIMEOUT

that's not an issue. issue is in query. but probably makes sense for consistency.

but any ideas why overwriting TIMEOUT had no effect?

• add_to_collection : Plex ReadTimeout Taxel/PlexTraktSync#1455 (comment)
• add_to_collection : Plex ReadTimeout Taxel/PlexTraktSync#1455 (comment)

[JonnyWong16]

Because plexapi is importing TIMEOUT as a symbol in the local module.

python-plexapi/plexapi/server.py

Line 9 in b783a56

from plexapi import BASE_HEADERS, CONFIG, TIMEOUT, log, logfilter

https://stackoverflow.com/a/3536638