search

uhd-urz / elAPI

An extensible API client for eLabFTW
GNU Affero General Public License v3.0
5 stars 0 forks source link

Occasional ReadTimeout #39

Closed alexander-haller closed 1 month ago

alexander-haller commented 1 month ago

In GitLab by @mhxion on Jun 9, 2024, 14:39

First, I have encountered ReadTimeout many times (though it did not happen in way that I could debug). Then again, the ReadTimeout occurred during @alexander-haller's HeFDI talk.

From HTTPX documentation:

The read timeout specifies the maximum duration to wait for a chunk of data to be received (for example, a chunk of the response body). If HTTPX is unable to receive data within this time frame, a ReadTimeout exception is raised.

We might have to increase the timeout from default 5s. Right now, elAPI doesn't allow modifying/overloading some of these basic client parameters via its APIs. We could also allow such modifications (related to #20).

alexander-haller commented 1 month ago

All low Prio.

We also might want to investigate why it takes > 5000ms for our server to respond.

On a generell note I like the option to modify timeouts for users (might be useful for researches in the field)

alexander-haller commented 1 month ago

In GitLab by @mhxion on Jun 13, 2024, 22:01

Closing this issue as the timeout issue could be just a server response delay. If not, i.e., if the ReadTimeout keeps happening with all our servers (it should not happen with dev-002), we will have to be up for a thorough debugging session. For now, --timeout 15 will solve any timeout related error.