Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timeout error (FI-1708) #248

Open
madhavdarji opened this issue Sep 2, 2022 · 8 comments
Open

Timeout error (FI-1708) #248

madhavdarji opened this issue Sep 2, 2022 · 8 comments
Labels
feature-improvement This is a new feature or improvement to be added to Inferno inferno-core This issue affects inferno-core

Comments

@madhavdarji
Copy link

Is it possible for Inferno to list test that failed due to timeout and continue with other pre-canned tests? Below is an error we are seeing in 1 of our tests.

4.15.01 Server returns valid results for Observation search by patient + code
Error: Timed out reading data from server

/usr/local/bundle/gems/rest-client-2.1.0/lib/restclient/request.rb:751:in `rescue in transmit'

2022-09-02 15_23_32-Inferno

@yunwwang
Copy link
Contributor

yunwwang commented Sep 6, 2022

@madhavdarji

Thank you for raising this issue to us. We will add more details to the error display.

@yunwwang yunwwang changed the title Timeout error Timeout error (FI-1708) Sep 6, 2022
@yunwwang yunwwang added will fix Issue is planned to be addressed in future release inferno-core This issue affects inferno-core labels Sep 6, 2022
@madhavdarji
Copy link
Author

What's the read timeout setup for single patient API?

@jpdow02
Copy link

jpdow02 commented Sep 29, 2022

Can you make this read timeout setting configurable? I understand the default is 60 seconds.

@yunwwang
Copy link
Contributor

yunwwang commented Oct 4, 2022

@jpdow02 What is the use case that a server could take more than 60 second to respond a FHIR request?

@satyaki-bhargava
Copy link

@yunwwang : As most of these FHIR servers will be test servers or mock server will have less power in terms of hardware. That is where we can use larger timeout values. However it will lead to a issue of token being expired within 59 seconds, so we are adding it, we also need to think of getting token refreshed for a given test case.

@yunwwang
Copy link
Contributor

"As most of these FHIR servers will be test servers or mock server will have less power in terms of hardware."
I am not sure if this is true. That also means server implementer has options to boost server performance.

If you do need an exception due to server limitation, please work with your proctor to figure out what your options are.

@yunwwang yunwwang added feature-improvement This is a new feature or improvement to be added to Inferno and removed will fix Issue is planned to be addressed in future release labels Apr 20, 2023
@yunwwang
Copy link
Contributor

Recategorize this issue. Inferno will provide a better test error message instead of the purple error as shown in the screen shot

@MCoderIndia
Copy link

MCoderIndia commented Sep 19, 2023

We have observed that the following socket error:
image

In this client socket timeout error, not sure, please look into it, as it seems related to the same issues hence adding here. Please let me know if need to create new issue. Observation the socket closed once 1000 resources have been read.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-improvement This is a new feature or improvement to be added to Inferno inferno-core This issue affects inferno-core
Projects
None yet
Development

No branches or pull requests

5 participants